Mobile first with bem and scss and javascript

Community feedback

• 

  eagermonument81•210

  @ixtk

  Posted about 4 years ago

  • header needs some black overlay on top because I can barely see white navigation links. It was in design too
  • the modal popup isn't centered and is very long, maybe it would make sense to add a scrollbar
  • button font-size is very small in modal
  • I would add cursor: pointer to the modal close button and radio checkbox section

  Also, check the report

  When it comes to folder structure, this from pro membership challenge files has become my new template

  - assets
    - mobile
    - tablet
    - desktop
    - shared (for icons that appear on all layouts, maybe logo.svg)
  - css
    - style.css
  - js
    - main.js
  .gitignore
  index.html
  

  11

  Marked as helpful
• 

  

  ApplePieGiraffe•30,525

  @ApplePieGiraffe

  Posted about 4 years ago

  Hello there, Heritier Akilimali! 👋

  Good job on this challenge! 👏 Your solution seems to work pretty well! 😀

  The only super tiny suggestion that I have is to perhaps add cursor: pointer to some of the interactive elements around the site (such as the pledge options and the close button inside the pledge popup). 🙂

  It also might be worth taking a look at your solution report and trying to clear up some of the errors that are there in order to improve the accessibility and semantics of your solution. 😉

  Keep coding (and happy coding, too)! 😁

  02

  Marked as helpful