QR_Code Solution HTML_CSS

Community feedback

• 

  Munsif Ali•200

  @Munsif-AliPosted over 1 year ago

  although your solution is looking good and your code is pretty clean but i would like to give some suggestions

  • try to create variable for the colors you are using in your CSS and then assign these variable to the property like this

  :root{
  --primary-blue-color:#1a75ff;
  }
  p{
  background-color:var(--primary-blue-color);
  }
  

  • you haven't used the font that was mention in the style guide.
  • also remove the opacity 80% from the heading if you can look to the design it is fully visible and you have make it 80% visible by providing opacity:80%; which is not looking good.
  • lastly i would suggest to make the background-shadow a little bit light its very strong now.

  Marked as helpful

  1
• 

  Munsif Ali•200

  @Munsif-AliPosted over 1 year ago

  your code repository link is not working its broken.

  Marked as helpful

  1

  ---

  

  Furqan Ahmed•30

  @Furqan5404Posted over 1 year ago

  @Munsif-Ali Updated the Link now.. Thanks..

  0