Skip to content
Submitted about 1 year ago

Responsive CSS Grid, Flexbox

Leke Adeloye•60
@Lekeadeloye
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Order summary component
  • HTML
  • CSS
1newbie

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
    @lrobb95Posted about 1 year ago

    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
    Leke Adeloye•60
    @LekeadeloyePosted about 1 year ago

    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
    @lrobb95Posted about 1 year ago

    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

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