Design comparison
SolutionDesign
Community feedback
- @Alex-Archer-IPosted 5 months ago
Hi!
Cool work, congrats =)
Here a few tips of how you can improve it.
- Use
min-height
forbody
instead ofheight
in case if the content will be larger than the screen. - It's better use
rem
values for font size. Those values depends on user's font settings, so they provides better UE.1rem = 16px
by default. - There's no need to wrap
h2
,p
and other tags withdiv
. You can assign them classes and style directly. The less tags, the better =)
Hope that helps. You are doing great, keep forward =)
Marked as helpful0@richxrdreisPosted 5 months ago@Alex-Archer-I Appreciate it man, ty soo much!!! I'm gonna keep trainning.
1@Alex-Archer-IPosted 5 months ago@richxrdreis
There is always something to train no matter of the experience =)
Good luck, and feel free to ask something if need =)
Marked as helpful0 - Use
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