Order Summary Card with HTML5 CSS3 SASS

Community feedback

• 

  rikvandersar•145

  @RikvanderSarPosted over 3 years ago

  Hi Adebayo,

  Look good to me! If I compare your version with the design it looks like the inactive and active state of the links are vice versa. And I wonder why you've used a psuedo element for the background image. Wouldn't a background image on the body work as well and be less code?

  Marked as helpful

  1

  ---

  

  Adebayo Abdulwaasi•40

  @AbduwaasiPosted over 3 years ago

  Thanks @RikvanderSar I really appreciate the time you took to review my code.

  This is my first time working on a design, I usually follow tutorial video, to be honest I didn't understand what the active state mean in the design till I see your review, I will implement the correction ASAP.

  For the background image I actually thought about inserting it in my background section but I was confused on how I will make the image appear on top of the background color. That's why I used pseudo element and gave it z-index

  0
• 

  rikvandersar•145

  @RikvanderSarPosted over 3 years ago

  If you give your body a background image and have a child element for your card the card will lay on top of your background image. But you have to set a height and width for you body to display to full image. I'd go with height: 100vh

  Marked as helpful

  0

  ---

  

  Adebayo Abdulwaasi•40

  @AbduwaasiPosted over 3 years ago

  @RikvanderSar okay I will give it a try

  0