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 using CSS Grid

Monica 260

@Monica-MR

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


Here's my solution for this challenge, feel free to write any suggestions. Thank You.

Community feedback

Vanza Setia 27,795

@vanzasetia

Posted

Hi, Monica! 👋

The card is filling the entire of my screen. Are you doing it on purpose?

Some feedback from me:

  • Accessibility
    • Don't use header for the card content since it is not a full webpage. This is one chunk of content that all belong together and in a real website would sit with other content.
    • I would recommend using the dollar sign ($) directly instead of using &dollar. The UTF-8 charset supports all symbols including emoji.
    • About the Why Us section, use ul and wrap each text with li instead of using p elements.
    • Create a custom :focus-visible styling to any interactive elements (button, links, input, textarea). This will make the users can navigate this website using keyboard (by using Tab key) easily.
  • Styling
    • Make the website looks closer to the design by decreasing the size of the card.
    • Don't limit the height of the body element, it will not allow the users to scroll the page if the page content needs more height. Use min-height instead.

That's it! I hope my feedback helps!

Marked as helpful

1

Monica 260

@Monica-MR

Posted

@vanzasetia Hi! I realised the card was not supposed to fill the whole screen when I uploaded the solution. I was confused about the design. Thank you so much for your feedback, there are really hepful for me.

0
Dev Rathore 2,600

@GitHub-dev12345

Posted

Congratulation to complete the challenge ❤️👍 My small suggestion : 1.In Card design CSS Code Used this:

transform : scale(0.8); this property decrease the size of card. 😉

large size for increase the number of scale & small size for decrease the number of scale

And in image card design give the opacity less number like :- opacity : 0.5, 0.8.

I hope you find this helpful

1
Naveen Gumaste 10,460

@NaveenGumaste

Posted

Hay ! Monica Good Job on challenge

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