@correlucas
Posted
👾Hello Karol, Congratulations on completing this challenge!
Great code and great solution! I’ve few suggestions for you that you can consider adding to your code:
1.Remove the 1440px
from the container, note that these sizes given in the style-guide.md
in the case 1440px for desktop and 375px are only a reference value to say in which dimension the design were created, don't stick to these values because your solution should be fully responsive and fit any screen size.
2.Its amazing that you’ve customized some interaction, you’ve to pay attention because when you add some hover effects
you've to think that for the mobile version there's an issue that the elements with hover get stuck
on the active state of the hover until you click on something else. Something you can do to avoid this problem on mobile, the stick hover, is to add a media query saying that you want hover effects only on devices that are not touch
. So you can use:
@media (hover: hover) {
{ADD HERE ALL THE CLASSES WITH HOVER EFFECTS TO DISABLE IT ON MOBILE}
}
✌️ I hope this helps you and happy coding!
Marked as helpful