Skip to content
Submitted about 3 years ago

Old Summary Card using HTML,CSS

SHAHAB MALIK•345
@Shahab-Malikk
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

Any kind of feedback will be appericiated

Community feedback

  • Nguyen Duy•245
    @koalalikecodePosted about 3 years ago

    Great work on this challenge bro, but I think you can have some regulations for a better job

    1. You should fix your body background with background-size: contain;
    2. If your div tag means main, you should use the main tag instead
    3. Your solution should be responsive for the mobile version.
    4. Your annual plan's height is too big compared to the design.

    Happy coding!

    0
  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    Great effort my friend. You should fix the background image and the selected plan container's height.

    0
  • Sami Maachi•230
    @Sami-21Posted about 3 years ago

    Great effort my friend. You should fix the background image and the selected plan container's height.

    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