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

Order Summary Card Solution

#accessibility#semantic-ui

@chiamaka-ugwu

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View Challenge

Design Comparison

SolutionDesign

Report

1

Accessibility Issues

0

HTML Issues

View Report

Community Feedback

Deniel Den 21,600

@denielden

Posted

Hi Chiamaka, I took some time to look at your solution and you did a great job!

Also I have some tips for improving your code:

  • add descriptive text in the alt attribute of the image
  • To make it look as close to the design as possible set width: 25rem; to .card-container class
  • remove all margin from body class because with flex they are superfluous
  • try to add a little transition on the element with hover effect
  • For add the top image in the background just put more specific background properties to the body:
background: url("../img/pattern-background-desktop.svg") no-repeat top center;
background-size: contain;
background-color: #e0e8ff;

Overall you did well :)

Hope this help and happy coding!

Marked as helpful

1

Abhik 4,740

@abhik-b

Posted

👋 Hello Chiamaka, Your solution is great & responsive enough . Good job 👍 I am just sharing some of my opinions that might make it look nicer.

  • use object-fit: cover; to img to make the image look responsive & nice
  • in card-container instead of width:30% use max-width:400px (use value of your own choice) can make the card look nice & responsive for large screens.

Other than that your solution is nice , Well Done & please keep coding amazing solutions like this 😃👍

Marked as helpful

1

Please focus on giving high-quality, helpful feedback and answering any questions chiamaka-ugwu might have. Here are some key points to consider:

  • Does the solution include semantic HTML?
  • Is it accessible, and what improvements could be made?
  • Does the layout look good on a range of screen sizes?
  • Is the code well-structured, readable, and reusable?

You can read our community guidelines if you're unsure what to post.

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!