Skip to content
Submitted over 3 years ago

HTML,SASS

tomas938•385
@tomas938
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

Another fun challenge, i have some problems building footer :D but i enjoyed it in the end. Feedback is welcome ;)

Community feedback

  • Benjo Quilario•1,810
    @benjoquilarioPosted over 3 years ago

    Congrats on finishing this one @tomas938

    It looks great and I like the opening animation a lot, My only suggestion is to add the main tag for html. The <main> tag specifies the main content of a document. The content inside the <main> element should be unique to the document. It should not contain any content that is repeated across documents such as sidebars, navigation links, copyright information, site logos, and search forms.

    For Image alt

    and also it also make a lot better if you add the :hover state on the footer link

    that's it good luck and keep coding!

    Marked as helpful
    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