@alberto-rj
Posted
👋 Hi, @mkostrikov! Congratulations on investing your time and leveling up with the challenge, Product preview card component
On line 76 of your internal style
You could just specify border-radius
from the upper corners
instead of all corners of the image to match the design on mobile device.
👎 Don't
border-radius: 1rem;
👍 Do
border-radius: 1rem 1rem 0 0;
On line 218 to 220 of your index.html
file:
Instead of using the h2
tag, you could use the h1
tag
because in this case it is in the correct hierarchical order.
Titles are in the correct hierarchical order when:
h1
is the main title;h2
is just the subtitle ofh1
;h3
is just the subtitle ofh2
;h4
is just the subtitle ofh3
;h5
is just the subtitle ofh4
;h6
is just the subtitle ofh5
;
Keep learning, coding and sharing! 🌟
Happy coding! 🚀
Marked as helpful