All comments

• 

  Ramachandran Varadaraju•30

  @ramachandran-varadarajuSubmitted over 1 year ago

  

  Responsive landing page using CSS Grid

  1

  10
  

  James Whatley•130

  @jameswhatleyPosted over 1 year ago

  Hi, great job on this challenge! One improvement that could be made is to centre the grid section in the middle of the page.

  An easy way to do this would be to set the height of the body to 100vh and then use flexbox to centre the component horizontally and vertically (using justify-content and align-items).

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Muhammad Noman•230

  @BitWizCoderSubmitted over 1 year ago

  What was difficult for you in this project? CSS or javascript?

  

  Interactive rating component

  1

  20
  

  James Whatley•130

  @jameswhatleyPosted over 1 year ago

  Good work, this looks great! Just something I noticed, it looks like you can select multiple ratings at the same time. So, if I wanted to I could select all 5 rating options.

  It might be worth reviewing your JavaScript code to check if any of the other rating buttons contain the "active" class and if it does then remove it, but only when another one has been selected. So in the end they behave like radio buttons.

  Well done on an excellent job though!

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  Esther•10

  @estherlardzeSubmitted over 1 year ago

  I didn't find any challenge as to this project is very basic

  

  QR-Code challenge using Html and Css

  1

  30
  

  James Whatley•130

  @jameswhatleyPosted over 1 year ago

  Good Job! I noticed the header wasn't aligned correctly. Make sure to use text-align: center on the heading, to align it to the center of its parent.

  0