Design comparison
Solution retrospective
Hey, guys !! this is my first developed challenge, i hope i managed to apply the correct concepts of HTML and CSS. Please, if I can improve anything, let me know!!! thankful .
Community feedback
- @dwhensonPosted about 3 years ago
Hey @Danny-for congrats on completing your first challenge! š
I think it looks good, and you've made good use of flex to align things nicely.
Some small suggestions:
- Maybe try a couple of different options on the middle flex container to see what layout matches - I wondered if
space-around
might work? Not sure though! - A neat trick to center components like this on the page it to set
display: grid
on the body, along withplace-content: center;
andmin-height: 100vh;
- this should help you with a few challenges
Keep up the good work! š
Cheers š
Dave
Marked as helpful5@Danny-forPosted about 3 years agoohh Thank you! @dwhenson, I will try to put into practice your suggestions/feedback on how to centralize the components.
success šš½ Thank you so much
- Danny
0 - Maybe try a couple of different options on the middle flex container to see what layout matches - I wondered if
- @hardy333Posted about 3 years ago
Hi, this is great result for the first time - good job.
Few suggestion:
- Make button full width of its container so that it will be the same size as
.container__plans
. - use smaller font-size on paragraph text
- Button hover state
background-color
is too gray, try to change it and addbox-shadow
also.
Marked as helpful2@Danny-forPosted about 3 years agohello @hardy333 ! , thanks so much for the feedback . I will try to improve it.
Thank you so much !
- Danny
0 - Make button full width of its container so that it will be the same size as
- @jones9411Posted about 3 years ago
HI @danny-for,
Good job on the solution, I've just done this one myself as well. The advice I was given was to use more semantic HTML tags like main, section, header, etc.
Also maybe use some more padding/margin to add a little bit more space between elements
Well done, Carl
Marked as helpful2@Danny-forPosted about 3 years agoHi @jones9411, thank you very much for the feedback I will try to put it into practice and congratulations for developing this challenge too.
Thank you so much !
-Danny
0
Please log in to post a comment
Log in with GitHubJoin 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