Responsive landing page for crowdfunding with SASS

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted over 2 years ago

  Hi

  This goes very narrow on mobile.

  You need to be more careful with how you're using viewport units here as the header bar background isn't deep enough on mobile landscape and the modal background doesn't reach the bottom of the screen when I scroll.

  It's also essential to trap focus and prevent background scroll when building modals. Scott o hara has written about this a lot, and Adrian roselli if you want to read more

  Marked as helpful

  1

  ---

  

  MOHAMAD NAZIR MD NAZRI•180

  @NAZIRwill29Posted over 2 years ago

  @grace-snow Thank you for the advice.

  Yeah, it really is problem for modal background. I should testing my web by browse it in the phone also, rather than using the laptop alone.

  0

  

  T

  Grace•31,270

  @grace-snowPosted over 2 years ago

  @NAZIRwill29 keyboard and screenreader as well

  1

  

  MOHAMAD NAZIR MD NAZRI•180

  @NAZIRwill29Posted over 2 years ago

  @grace-snow thank you. I would work on the alt and aria-label.

  0