html css js webpack

Community feedback

• 

  Avinash Dhauni•170

  @avinashdhauni

  Posted almost 3 years ago

  Hello Doris, I noticed a little problem with the project. When you click in between numbers, like for example- between 2 or 3, or maybe 3 or 4, etc then all the numbers are being selected. And after that, if you click submit then all the numbers are shown on the Thank You component. I don't know what the issue is because you seemed to have used webpack and other stuff that I am not familiar with. And also I guess you forgot to give hover effects to the submit button as well. But congratulations on completing the challenge. Happy Coding!

  11

  Marked as helpful
• 

  Elaine•11,360

  @elaineleung

  Posted almost 3 years ago

  Hallo Doris, well done on your second challenge! Like what Avinash mentioned, there's an issue with the previously "activ" numbers being still stuck in "activ" mode. To remove the class, try adding a forEach in the zahlAusgeben function to loop through the elements:

  function zahlAusgeben(e) {
      document.querySelectorAll('.choose__selection-n').forEach( btn => {
         if (btn.classList.contains('activ')) btn.classList.remove('activ')   
      })
      if (e.target && e.target.matches('div.choose__selection-n')){
          number = e.target.innerText;
          e.target.classList.add('activ');
      };
  };
  

  Another thing I recommend here is, instead of using div for the buttons, try button first, as this would be better for screen reader users since button can be controlled by a tab key, but a div cannot unless you write a bit more code to it.

  Viel Erfolg!

  00