@FluffyKas
Posted
Hi,
Hmm, I don't see any extra room for scrolling so I'm not sure what you mean... The best thing you can do is to center your component vertically and horizontally which you're already doing (although I'd change the min-height to 100vh, 90vh doesn't seem to make a lot of sense). ^^ Your component looks nice and it functions really well so I'm just going to note a few smaller details where you might wanna improve:
-
All images must have an alt text, even decorative images. Leave the alt text empty if it's decorative (in this case it is) and add an aria-hidden=true as well.
-
Don't set font sizes in pixels. If a user ever changed their browser settings, your solution would not scale with it. Use rem, for example.
-
For your buttons please use the button element. Paragraphs probably shouldn't go inside lists anyway (I suppose it's not strictly wrong, it just doesn't make a lot of sense).
-
Adding a
cursor: pointerto all buttons would be great.
I hope this was helpful, good luck! ^^
Marked as helpful
@pmork7
Posted
@FluffyKas
Hi,
Thank you! I appreciate your feedback. I made some of the changes you suggested. I appreciate you taking the time to give me these suggestions. This was extremely helpful.
