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

@Olarra18

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


I am a beginner to programming, all feedback are welcome but I will glad if you can check through my html, css, js code structure and rate my coding skill and feedback on where adjustment can be made.

Community feedback

@WDWaleed

Posted

The overall layout looks good but you've done a major problem with the colors.

The text isn't readable so try to change the colors as shown in the style guide. It will make the card look much better.

Also the very first heading "Join our Commmunity" should have a larger font-size. You should write this inside of an h2 element because this is an important heading.

To make the card look like a "card" use box-shadow property. Here is what I used:

box-shadaow: 1px 3px 15px rgba(0, 0, 0, 0.2);
0
_nehal💎 6,710

@NehalSahu8055

Posted

Hello Coder 👋.

Congratulations on successfully completing the challenge! 🎉

Few suggestions regarding design.

➨To properly center the container.

  • USING FLEXBOX
body{
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}
  • USING GRID
body{
min-height: 100vh;
display: grid;
place-items: center;
}

➨ Give some width or more specific max-width to your card as it is get very large on larger screen.

➨ Use responsive units(rem, em, %) from next project. Explore respective use cases on google.

link.

➨ You can use accessibility features like aria, sr-only, and title which are accessible to screen readers.

I hope you find this helpful.

Happy coding😄

0

@Olarra18

Posted

@NehalSahu8055 Thanks for the feedback, I really appreciate it and I'm grateful. I will update my code file with your suggestion, once again thank you.

1
_nehal💎 6,710

@NehalSahu8055

Posted

@Olarra18

Tip:

  • Generate new screenshot after making the necessary changes.
0

@Olarra18

Posted

@NehalSahu8055 Thank you so much for the super amazing CSS tip you showed me, it makes my project look amazing.

1

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