Responsive Single price grid component

Community feedback

• 

  Matias Ludueña•1,990

  @matiasluduena23Posted 12 months ago

  Hi Dalha’s! Good work! Just one advice that might help you!

  • If you notice in your <a> tag hover effect the height or your card jump. This is because you add border to your <a> . To avoid this you can add .signup-btn { border: 1px solid transparent} . Your border is already there, but transparent so avoid the jump.

  Good code!

  Matias

  Marked as helpful

  1

  ---

  

  Dalha•90

  @dalhaldalhaPosted 12 months ago

  @matiasluduena23 thanks for the tip, I'll fix it up.

  0
• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted 12 months ago

  Hello there 👋. Good job on completing the challenge !

  I have another small suggestion about your code that might interest you.

  • For the element below the 'Why Us' section, consider using an unordered list (<ul>) instead of a paragraph. This is because if a screen reader reads this paragraph, it will read everything continuously, making it harder to understand.

    It should look like this:

    <ul>
      <li>Tutorials by industry experts</li>
      <li>Peer & expert code review</li>
      <li>Coding exercises</li>
      <li>Access to our GitHub repos</li>
      <li>Community forum</li>
      <li>Flashcard decks</li>
      <li>New videos every week</li>
    </ul>
    

  Happy coding!

  Marked as helpful

  0

  ---

  

  Dalha•90

  @dalhaldalhaPosted 12 months ago

  @MelvinAguilar Alright thanks for the tip. Highly appreciated.

  0

  

  Dalha•90

  @dalhaldalhaPosted 12 months ago

  @MelvinAguilar btw how the hell u got 54k points!!

  0