P
Remus D. Buhaianu• 3,145
@Miculino
Posted
Congrats on completing the challenge, @Schmelzofen
Your solution looks really nice and it's responsive as well. Just a few suggestions based on what I've seen in your final solution:
-
The share menu doesn't work as expected. There are multiple ways to approach this feature. You can create the share menu, hide it with
display: noneand then use JavaScript to toggle a class that'll hide / reveal the menu -
On smaller screen sizes, consider adding some horizontal
paddingon the .details element
Marked as helpful
1
Schmelzofen• 190
@Schmelzofen
Posted
Thanks for your input! :)
I think I fixed the share menu, but it only works on the desktop version :/
0
