Mobile first approach

Community feedback

• 

  Sam Guliker•540

  @Sam-GulikerPosted almost 3 years ago

  Hi, nice solution youv'e got there.

  I think you should go a head and take a more difficult challenge :)

  Happy coding

  0

  ---

  

  Ted•180

  @sharipoff-0-1Posted almost 3 years ago

  @Sam-Guliker Hi, thanks, man :)

  0
• 

  Akshay•810

  @akshaywebsterPosted almost 3 years ago

  Hi, Ted.

  Great work on the project.

  A couple of things to notice:

  • There's no background color for the body that takes the lower half of the screen other than the curved SVG image, so you can apply the following style to fix the issue:

  body {
    background-color: hsl(225, 100%, 94%);
  }
  

  Other than that, it's all good. Keep up the good work.

  I hope my feedback is helpful to you. Have a great day! :)

  0

  ---

  

  Ted•180

  @sharipoff-0-1Posted almost 3 years ago

  @akshaywebster Hi, thank you, it is helpful :), have a nice day.

  0
• 

  Rimka•600

  @karimfilaliPosted almost 3 years ago

  Hello, the card looks really great, no accessibility issue or html issue ! For the background, I used the CSS property: background : url("images/..."), #color. Great job, though

  0

  ---

  

  Ted•180

  @sharipoff-0-1Posted almost 3 years ago

  @Rimka93 Hello, thank you for your feedback...

  0