Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Huddle-landing-page-with-alternating-feature-blocks

@Waldoozki


Design comparison


SolutionDesign

Solution retrospective


I think the code can be improved, right? Comment your suggestions, please.

Community feedback

Vanza Setia 27,795

@vanzasetia

Posted

Hi there! 👋

Some suggestions from me.

  • Add a <title> inside the head. The title of the page is the first thing that users will see before the site is fully loaded. Also, it is often the information that assistive technology will announce first. So, give your website a title.
  • figure element doesn't need to wrap every img element. The only reason to use figure is if you need to include a figcaption. Otherwise img tag is fine.
  • The styling for the .nav__cta.cta element is complicated. So, I recommend simplifying it.
    • Set the element to display: inline-block to make it easier to style (because by default a element is an inline element).
    • After that, you can use padding to give some width and height or simply make the button bigger. So, there's no need for width and height.
    • The hover effect should exist on all screen sizes. I notice that on 807px width the button doesn't have any hover effect.
    • Try to make the button looks closer to the original design. The background color is different from the design.
  • It looks like the styling of the interactive elements are complicated because of the :hover effect. If you can't make it simpler then I think it's okay.
  • Alternative text for images should not be hyphenated (not like code). Also, not all images need alternative text (e.g. illustrations).

That's it! I hope this helps!

Marked as helpful

0

@Guilherme-Verrange

Posted

Very good Axl, congratulations. A tip would be to change the color of the cards (grow, flowing and your users), because the colors of the background of the screen and of the cards are practically the same, and the shadows on top of the cards do not exist, which ends up getting confusing and disturbing the user experience, a tip is to follow the color lists that you have in the document itself when it is downloaded. And again, congratulations!!

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord