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

React + sass + flex

E.sa 300

@E-sa

Desktop design screenshot for the Todo app coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
3intermediate
View challenge

Design comparison


SolutionDesign

Solution retrospective


I get "All page content should be contained by landmarks" warning. Any Idea how to fix it?

Community feedback

@tesla-ambassador

Posted

Hey, this is a pretty awesome solution. I like it.

  1. if you want all page content to be contained within landmarks, you should make sure that all your "divs, image tags, buttons, forms... etc." are contained in landmarks like "Section, article, main, footer... etc." to enable people that are using screen readers to access your site.
  2. I think there's some sort of bug when I choose clear completed because it also clears those that I haven't marked as completed.
  3. Also make sure that all your "li" elements are contained within "ul" i.e. (<ul><li></li></ul> and on the same note, you can't have a div as a child element of an li.

4.Make sure that you only have one main landmark. I recommend that you replace your second main landmark with a section or article. 5. Remember to add labels to form elements to help people that use screen readers to access your site. Otherwise, I think this is a very good solution! Keep up the good work and keep on coding! Kudos!

Marked as helpful

1

E.sa 300

@E-sa

Posted

@tesla-ambassador

I try to do as you said for the land mark, thanks

I know its <ul><li>, but i made a dumb mistake and put div inside <ul> and i got warning for that so i changed <ul> to <div> (fast and lazy solution, i will fix it ;) )

Im gonna fix clear completed too.

And again thank u for you time checking my code and writing this comment i appreciate it. :)

0

@tesla-ambassador

Posted

@E-sa You're very welcome! Am glad I could help

1

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