Skip to content
Submitted over 3 years ago

Mobile First ~ HTML/SCSS

GerLC•455
@GerLC
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Fylo dark theme landing page
  • HTML
  • CSS
  • JS
2junior

Design comparison

SolutionDesign

Solution retrospective

Feel free to give a review!!

Community feedback

  • Andres Villalba•205
    @VILLALBA-22Posted over 3 years ago

    Everything is fine but you could use transitions when done hover an item

    1
    GerLC•455
    @GerLCPosted about 3 years ago

    @VILLALBA-22 Thanks for the review, yes it is an option to use transitions.

    0
  • Chamu•13,130
    @ChamuMutezvaPosted over 3 years ago

    Well done

    • site is responsive - checked on mobile
    • the email validation is perfect
    • for header links that should be more accessible in an unordered list
    • alt values must be descriptive to aid assistive technology users to visualize the message that is being put across. Decorative images should have an empty alt value. Values like hero images are shallow
    1
    GerLC•455
    @GerLCPosted about 3 years ago

    @ChamuMutezva Thanks for the review.

    0

Please log in to post a comment

Log in with GitHub

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