Order Summary Card using TailwindCSS

Community feedback

• 

  Hania B.•1,320

  @techantherePosted almost 3 years ago

  Hey, your solution looks good except I see the semantics have been totally ignored in the code which is not a good thing at all. You should definitely give it a try to resolve the issues in the report. Other than that the card is not centered vertically so adding some vertical padding on the body will center it. Another thing is it's breaking on smaller screens might be tuning the card width for smaller screens should work. Everything else's fine :)

  1

  ---

  

  jul_julham•210

  @zulhamsyPosted almost 3 years ago

  @techanthere, I added semantics markup as you suggest, now accessibility issues are gone, thanks!

  btw, I try with small screen (393px) and its still looks good and not breaking at all, what resolution are you using?

  0

  

  Hania B.•1,320

  @techantherePosted almost 3 years ago

  @zulhamsy great to hear you have resolved accessibility issues, for size so that's the reason it's visibly fine till 400px on dev tools but I have checked on my phone it's breaking there. It means that for screen sizes smaller than that it starts breaking it's good to consider smaller screens up to 320px.

  Marked as helpful

  0