Submitted

First mobile and flex-box

@azyepes29

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View Challenge

Design Comparison

SolutionDesign

Report

3

Accessibility Issues

0

HTML Issues

View Report

Luis Yps Fdez’s questions for the community

I really felt better and more confidence with this my second challenge. Thanks for all comments in the first one and welcome to the new comments in this one.

Community Feedback

P

@Eric-Ferole

Posted

Well done Luis. It looks pretty much the same as the solution. Your HTML class are well written, but you could use BEM methodology to write your CSS.

Keep up !

Marked as helpful

1


@azyepes29

Posted

@Eric-Ferole Noted. Thanks Eric.

0

darryncodes 5,170

@darryncodes

Posted

Hi Luis,

Awesome solution, really well done!\

You could clear down your accessibility/html report if you use some more descriptive elements. Swap <div class="main"> for <main class="main"> and <div class="attribution"> <footer class="attribution">. Semantic html is really important for accessibility and helps with seo.

All the best!

Marked as helpful

1


@azyepes29

Posted

@darryncodes Thanks, I'll apply it next time!

0

Give some feedback to @azyepes29 about their solution...

Slack logo

Join our Slack community

Join over 100,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!