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

Single price grid component

Mario 180

@bembem33

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any suggestions are welcome! Thank you.

Community feedback

Rio Cantre 9,690

@RioCantre

Posted

Hello there! Awesome job with this one! Viewing your solution you did well with the design. I would only suggest the following for you...

  • Change the color value of the background in .bottom__right into background: hsla(180, 39%, 56%, 1);
  • Change the font color of the .bottom__right p into color: var(--light-gray);
  • Wrap the container with specific tag like main
  • Add a padding to the attribution in smaller view

Hope this helps and Keep up the good work!

Marked as helpful

1

Mario 180

@bembem33

Posted

@RioCantre Hello... Thank you! This is what I looking for background: hsla(180, 39%, 56%, ### 1 ###); , I try to find the color by myself but without a success.

1
Rio Cantre 9,690

@RioCantre

Posted

@bembem33 Don't mention it, if your using google chrome there is an extension called ColorZilla, you can download it and use it to pick colors.

Marked as helpful

1
Naveen Gumaste 10,460

@NaveenGumaste

Posted

Hay ! Good Job you made it look nearly perfect to the preview

These below mentioned tricks will help you remove any Accessibility Issues

-> Add Main tag after body like it should be your container

-> For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc

-> But use header tag only once in main heading element.

-> Always use h1 first and then h2, h3 and so on

Keep up the good work!

Marked as helpful

0
Abhik 4,840

@abhik-b

Posted

👋 Hello Mario , Your solution looks absolutely amazing on large screens & the card looks very responsive.

Some of my opinions to make it look nicer in small screens as well :

  • use this styles in body
justify-content: center; */ remove this line it is of no use */
    min-height: 100vh;   */ so that for small screens body's height can increase*/ 
    grid-template-rows: 1fr auto; */ so that container & footer can be displayed without position relative*/
    gap: 2em; */ so that footer can have some spacing from the container*/
    padding: 4em 0; */ so that container can have some spacing from top without  using position relative & top*/
  • remove the max-height from container
  • remove position : relative & top from container & footer

Please keep up with this amazing work 👍

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

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