Sunnyside agency landing page - Grid, JavaScript, Flexbox

Community feedback

• 

  Aruj Joshi•290

  @gtalinPosted over 1 year ago

  The solution is very good. Like the animation for the down arrow.

  Since I have recently completed this challenge, I can provide a few suggestions:

  You can make the hero image (orange) 100vh so that we cannot see the next section. That would make the animated down arrow more relevant.

  In the features section where there is a text and image side by side, you can add a max width to the text and the text heading as well (just like you have added for the text displayed over background image). That way the text will not stretch out when we view the page on larger screens.

  Since you have already included elements for accessibility in your solution (links with a hidden span for people with assistive technologies) it might interest you that some people have problems with large animations on page. And smooth scroll is one of those. They can set a preference in their browsers prefers reduced motions. So you can enable smooth scrolling only for people who have not set a prefers reduced motion to true.

  I have found a useful link for it. Hope you find it helpful as well.

  Marked as helpful

  1

  ---

  

  Hoshiko•180

  @oliwiakrammPosted over 1 year ago

  @gtalin

  Thanks for your feedback 😁 I will take your advice and change it and thanks for the article I have seen articles about prefers reduced motions before but I forgot about it.

  0