html, css

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey! It looks good!...

  to make the annual plan and $59.99/year in the same line with music icon and change:

  .box2 {
      //...
      display: flex;   /* center with flex */
      align-items: center;
  }
  .music-icon {
      margin-right: 20px;
     /*margin-top: 10px; */   <--- Remove
     /*  height: 50px;  */   <--- Remove
     /*  width: 50px;   */    <--- Remove
  }
  .box3 {
     /* display: inline-block;  */    <--- Remove
     /* margin:10px 0 0 10px ;  */    <--- Remove
  }
  

  and you need to change the id attribute to class because you style the img element with class but you used the id attribute <img src="images/icon-music.svg" alt="music icon" id="music-icon">

  <img src="images/icon-music.svg" class="music-icon" alt="music icon" >
  

  • This <p class="Change">Change</p>, Should be an anchor link <a>
  • These <p class="payment">Proceed to Payment</p>, <p class="cancel-order">Cancel Order</p> Should be wrapped with <button>... Buttons are used for actions like opening or closing something, or sending a form.

  I hope this is helpful to you... Keep coding👍

  Marked as helpful

  0

  ---

  

  Pratyusha Majumder•280

  @FloraBloombluePosted over 2 years ago

  hi just a question, so I was doing a challenge where there is an image in black and white and I have to use a color over it. so the image with black and white has shade of color on it. I saw the overlay method, linear gradient. Maybe I am doing it wrong, but the image tag in html is there. Now for overlaying I am using that image class, and there I wrote the attributes like linear gradient.... etc. or box shadow, or overlay, it is not working. No color is coming over the picture. Can you guys help me on it?

  0

  

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  @FloraBloomblue Sure

  0
• 

  Rehman•280

  @Abdurehman420Posted over 2 years ago

  hello !! nice work there bro

  no external CSS is cool.in bigger projects you will be working with external CSS because writing everything in one single file makes it harder for others to understand

  if you used display flex on the music icon and text container, then give the 59,99$ paragraph, justify-self:flex-start;

  Marked as helpful

  0