Responsive Single Price Grid Component | HTML & CSS

Community feedback

• 

  P

  tediko•6,680

  @tedikoPosted 11 months ago

  Hello @ArdaBozan! Good job on this challenge! Here's my few suggestions:

  • Don't separate your HTML class names with "|". Separate the class names with a space, e.g. <div class="class1 class2">.
  • I'd use only one paragraph for your .price and .opacity text. Wrap opacity text into span element and style it. This will also get rid of .price-block div.
  • Your <button> element should be <a> anchor. Links take the user to a new location, such as a new web page or new section of the current page, buttons trigger some action, such as showing content on the page that was previously hidden, playing a video, or submitting a form. To me "Sign up" will take user to new page.

  Have fun!

  Marked as helpful

  1

  ---

  

  Arda Bozan•160

  @ArdaBozanPosted 11 months ago

  @tediko

  Thanks for the advice! The reason I separate classes in that way is because I heard it from someone experienced who said it didn't cause any issues, and the reason for separating them is to distinguish between a class I've previously created and a new one I'm going to create. And yes, yes :D I didn't establish the link because it's just a design.

  0