3-Theme-Responsive-Calculator

Community feedback

• Account deleted

  Everything in your solution looks ok and the functionality is fine but you can no longer user the period once you press reset... it's weird & I don't know why it's happening.

  Marked as helpful

  1

  ---

  

  Bartek•400

  @imxbartusPosted about 3 years ago

  @thulanigamtee thanks, i will fix that

  0

  

  Bartek•400

  @imxbartusPosted about 3 years ago

  @thulanigamtee i think now it works correctly.

  0
• 

  nmorajda•945

  @nmorajdaPosted about 3 years ago

  Cześć :) Wygląda dobrze ale kod JS jest moim zdaniem trochę do przerobienia.

  Chodzi mi o to, że zmian wyglądu (theme) powinna być opisana w pliku CSS, a JS tylko obsługiwać zdarzenie i np. dopisywać nazwę klasy w elemencie body. Cała resztę powinien być w pliku CSS.

  Na przykład:

  
  ::root {
    --body-bg: #e5e5e5;
  }
  
  .dark {
    --body-bg: #212121;
  }
  
  body {
    background: var(--body-bg);
  }
  

  a cały js to coś takiego:

  
  btn.addEventListener('click', () => {
    document.body.classList.toggle('dark')
  })
  

  Pozdrawiam, N

  Marked as helpful

  0

  ---

  

  Bartek•400

  @imxbartusPosted about 3 years ago

  @nmorajda Dzięki, z pewnością skorzystam przy najbliższym projekcie.

  0