QR code component challenge on Frontend Mentor

Community feedback

• 

  P

  Leslie•110

  @DereemiiPosted over 1 year ago

  Great work! Only I would change the class name from the footer from "attribution" to footer instead, because if I go directly to the CSS files, I won't get a clue about which element is.

  Marked as helpful

  0

  ---

  

  Maksudur Rahman•50

  @SparrowHawkEyePosted over 1 year ago

  It was already present in there Given file. thats why i didn't chanage it. but i used the semantic tag for the footer 😊. @Dereemii

  0
• 

  Don Delice Dushime•150

  @dondelice257Posted over 1 year ago

  Perfect job, but you can try to increase the border radius of the container with some px

  Marked as helpful

  0

  ---

  

  Maksudur Rahman•50

  @SparrowHawkEyePosted over 1 year ago

  Thanks a Lot :)@dondelice257

  1

  

  Don Delice Dushime•150

  @dondelice257Posted over 1 year ago

  @SparrowHawkEye , You are welcome, so please mark my feedback as a helpful one if it's

  0