Kamasah-Dickson• 5,610
@Kamasah-Dickson
Posted
- Your solution looks great but not responsive. Use max-width for your card instead of just width .
- To help with the accessibility issues wrap your card in a main semantic tag.
- Bonus Headings must be used in reducing order e.g h1,h2,h3,h4 do not mixed them up they should be used in descending order.
- Disable the next and prev button when there is a modal
- Add a margin between your header and the grid section. its too closer to the header and there is not enough space there.
- There should be a heading before a <section> tag. Don't use a section tag if there won't be any immediate heading.
I hope this was helpful.
Happy coding and have a nice day👍
Marked as helpful
0