@buneeIsSlo
Posted
Hey! @kxnzx, Amazing work on this challenge. It's nice to see how much you've learned through this challenge alone. With that being said, I feel some parts of the solution need changes:
-
The slide-in menu on the tablet view is a bit too wide. This is happening because you're using
%
to set the width. A quick fix for this problem would be to set a maximum width, say:max-width: 250px
; -
Hovering over the "Learn more" link creates a UI shift. This happens when you set a
border
on an element that didn't have it originally. Again, a very simple fix for this would be to use anoutline
like so:outline: 2px solid white;
. Also, check out this video to learn about the differences betweenoutline
andborder
.
Hope this helps :)
P.S: Using the details
and summary
tags to create the dropdown menu is pretty smart. I'm currently working on this project and am a little sad that I didn't come up with it myself. Keep Coding!
Marked as helpful
@kxnzx
Posted
@buneeIsSlo Thank you so much for the constructive feedback! Much appreciated :)