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

Responsive CSS Grid, Flexbox

@Lekeadeloye

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Please can I have some feedback with regards to making my HTML more accessible.

Also some advice on how I could have used more semantic HTML.

Any feedback on my CSS would be appreciated also.

Community feedback

lrobb95 200

@lrobb95

Posted

Hello!

First, I’d like to say awesome job on this project!

Now, I’m just a small fish in the pond, but for the accessibility part, I’d recommend putting a title, role, and/or a type on your buttons. Aria attributes are nice things to have as well. I know I could use those a lot more often myself.

Marked as helpful

0

@Lekeadeloye

Posted

Thanks for your feedback. Yh, i need to use Aria and roles a lot more. Im reading up on hiw tobmake your web pages more accessible. @lrobb95

1
lrobb95 200

@lrobb95

Posted

No problem, I’m happy to help! :)

Enjoy the learning!

P.S. Now I’m sure that almost everyone on this website knows of Kevin Powell, but if you don’t, he is one of the best resources I can think of and he talks quite a bit about accessibility.

I also would recommend videos from Kevin (Webdev Simplified). Both are really great at frontend development and even better at teaching it! @Beezy100

Marked as helpful

0

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