drag and drop mobile polyfill , localstorage api.

Community feedback

• 

  Antoine C•1,210

  @mattari97Posted over 2 years ago

  Hello.

  I would recommend to use css variables to toggle the theme.

  You would use something in js like :

  function toggleTheme() { document.documentElement.classList.toggle("darkmode") }

  Then in css at the top of the stylesheet:

  --color-1: #fff;

  --color-2: #333;

  --background-color: var(--color-1)

  .darkmode { --background-color: var(color-2); }

  body { backgroud-color: var(--background-color) }

  Like this you dont have to duplicate the whole stylesheet which make it painfull if you need to track changes on both everytime.

  Peace

  Marked as helpful

  0

  ---

  

  Oladejo Eniola•150

  @niol08Posted over 2 years ago

  @AntoineC-dev thanks for the feedback. I'd try to implement this method next time.

  0
• 

  Divine Obeten•2,415

  @Deevyn9Posted over 2 years ago

  Hi Ola, great solution you have here. About your question, normally it is said that you shouldn’t tweak it as long as it works 😃. However, one of the most important things to consider when building is speed optimization, and after checking your solution, it doesn’t appear to have any issues with it speed, so it doesn’t appear to be wrong.

  Happy coding 🎈

  Marked as helpful

  0

  ---

  

  Oladejo Eniola•150

  @niol08Posted over 2 years ago

  @Deevyn9 thanks for the feedback.

  1