Landing page using css grid

Community feedback

• 

  Alberto Raúl José•610

  @alberto-rjPosted 8 months ago

  👋 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;
  

  Learn more here

  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 of h1;
      • h3 is just the subtitle of h2;
        • h4 is just the subtitle of h3;
          • h5 is just the subtitle of h4;
            • h6 is just the subtitle of h5;

  Learn more here

  Keep learning, coding and sharing! 🌟

  Happy coding! 🚀

  Marked as helpful

  1
• 

  P

  tinnq66961996•280

  @tinnq66961996Posted 8 months ago

  Ok

  0