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 - using scss and Grid

@c-tregunna

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


Great little refresh for using Grid.

I am not sure if my font, especially the colors and weights, is a good match.

Feedback, as always, is welcomed in any form

Community feedback

@pikapikamart

Posted

Hey, great work. The content seems bigger but preferences right.

In terms of layout, desktop is fine and it resizes well when going in mobile state, thought it just get very narrow when almost reaching the breakpoint.

One concern is that, the learn more button, you should not use div for that. Since that learn more is supposed to be a anchor to redirect a user, you should use a tag for it and not div. Using a tag will make your markup more semantic. Changing that will be really good

Apart from those mentioned, good work^

0

@c-tregunna

Posted

@pikamart Thank you. You are right, not sure how I missed using the <a> tag. Will get that altered and look again at the layout, as realized it was a little bigger than intended.

Thank you for the feedback

0

@c-tregunna

Posted

I now realize I need to reduce some of the margins on my headers, this will help with the layout a little better I think. Will get that done shortly

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