Rostyslav Nazarenko• 560
@rostyslav-nazarenko
Posted
Hi! Amazing solution! 😍
I will try to provide feedback, but it is mostly tiny bits here and there.
- It is not recommended to set `html { font-size: 62.5%; } read more about it here. I'd recommend reading this entire article.
- Change
height: 100vh
tomin-height: 100vh
. If users set a slightly bigger font size they wouldn't be able to scroll and see the whole component. - Look at reports. In a real project, you wouldn't use
h1
for this but here it is a stand-alone project. Removep
element frombutton
. - Use
em
for media queries too.
Have a nice day!
Marked as helpful
1