Skip to content
Submitted almost 3 years ago

simple css

Narender Singh•295
@sinredneran
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Fylo data storage component
  • HTML
  • CSS
2junior

Design comparison

SolutionDesign

Solution retrospective

give me feedbacks.

happy new year.

Community feedback

  • Sam Guliker•540
    @Sam-GulikerPosted almost 3 years ago

    Hi,

    Clean solution you've got there. The only thing I could give feedback on is: Use landmarks. For example, a main, header, footer etc. More about this topic can be found here: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/main

    Happy coding and a good new year

    Marked as helpful
    1
    Narender Singh•295
    @sinredneranPosted almost 3 years ago

    @Sam-Guliker yeah, i know this problem and i will use these accessibility tags in future projects.

    thanks

    0
  • Anosha Ahmed•9,300
    @anoshaahmedPosted almost 3 years ago

    To get rid of the accessibility/HTML issues shown in your Report:

    • wrap everything in your body in <main> OR use semantic tags OR give role="" to the direct children of your <body> ... Click here to read more
    • have at least one <h1> in your code

    Good job! :)

    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