Responsive Layout using Flex-box

Community feedback

• 

  faizan•2,420

  @afaiz-spacePosted over 2 years ago

  Hey @Isaaclkm, congratulation on completing the challenges. I noticed in the b-price class color is gray so removed the bottom padding and added verticle-align:6px; remove the flex from the box class.

  Marked as helpful

  0
• 

  Rohit Kumar Saini•270

  @rockingrohit9639Posted over 2 years ago

  Hey @Isaaclkm, It is good to have classes on all of your HTML elements, but there are some CSS Selectors which are very helpful while working and styling using CSS. To learn about different CSS Selectors you can go with the following link. CSS Selectors

  Apart from it, some issues which I found are:-

  • You should use semantic HTML tags like main, section, footer, etc in your HTML.
  • You should use / in image src not \ E.g. Use this

  <img src=`images/perf1.jpg` alt=`perfume bottle`>
  

  not this

  <img src=`images\perf1.jpg` alt=`perfume bottle`>
  

  Marked as helpful

  0