Single price challenge responsive grid media query mobile first

Community feedback

• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 years ago

  Hello mariano ! Congo 👏 on completing this challenge

  Let's look at some of your issues, shall we:

  • The background color is given in style-guide.md

  • Why us section color is too much light and i know it was not given in style-guide so use color-zilla chrome extension. and you will get the color

  • This is color i used for my solution --light-cyan: hsl(179, 45%, 52%); check my solution here if needed

  happy Coding😀

  Marked as helpful

  0

  ---

  

  mariano•60

  @marianojugoPosted over 2 years ago

  @Crazimonk Thanks i didnt know color zilla. its very helpful 👍

  1
• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 years ago

  Hey marianojugo good job completing this challenge, keep up the good work

  Your solution looks great however I think that the title Join our community needs a bit more margin on the top. You can achieve this using the margin-top property.

  In terms of your accessibility issues, simply wrap all your content between the semantic <main> tag. You can also practice using semantic tag henceforth in your code. You can read more about semantic tags here

  https://www.w3schools.com/html/html5_semantic_elements.asp

  I hope this helps

  Cheers, Happy coding 👍

  Marked as helpful

  0

  ---

  

  mariano•60

  @marianojugoPosted over 2 years ago

  @TheCoderGuru Thanks for your advice 👍

  0
• 

  Kacper Kwinta•1,405

  @kacperkwintaPosted over 2 years ago

  Hi! Looks great!

  Some tips:

  • Page should contain a level-one heading

  Marked as helpful

  0
• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  Fabulous Work Mariano..

  0

  ---

  

  mariano•60

  @marianojugoPosted over 2 years ago

  @Samadeen thank you

  0