All comments

• 

  JoshuaWBlick•270

  @JoshuaBlick94Submitted about 2 years ago

  Hello guys, how did I do on this?

  

  Intro component with sign-up form

  1

  10
  

  Vincent Chen•120

  @VincentChen6345Posted about 2 years ago

  Hey Josh, a couple of things I noticed:

  1. when the screen is in mobile mode, the background image doesn't cover all the content.
  2. the box shadow is different to the design
  3. your styling of the active input field border is blue whilst the styling in the design is black
  4. Im assuming you're learning JS to do the validation functionality?

  Otherwise, looks good!

  0
• 

  Abdel!•130

  @Abdel-2020Submitted almost 3 years ago

  

  Features Page made with Flexbox and Mobile First Workflow

  1

  20
  

  Vincent Chen•120

  @VincentChen6345Posted almost 3 years ago

  Hey Abdel!

  It looks like your responsive layout isnt designed from 427px to 1440px. You might want to give this screen width range a responsive design so it looks more complete.

  Goodluck!

  1

  This comment has received 2 repliesShow
• 

  Jociel•90

  @jsmeyringSubmitted almost 3 years ago

  Could you guys check my code and give me feedback about Grid Layout? Thanks.

  

  Responsive Sigle Price landing page using CSS Grid Layout

  2

  20
  

  Vincent Chen•120

  @VincentChen6345Posted almost 3 years ago

  Hey man! Im a newbie myself so good on you for finishing this.

  • there are some typos in your code that you can fix up like 'conteiner'
  • consider adding media queries for widths greater than 450px as well because you dont want the card to be cut off from the sides on a smaller screen
  • the 'opacity' or 'color' of the texts can be adjusted to more accurately depict the original design (e.g. in the 'why us' section, the text is lighter)

  Goodluck!

  Marked as helpful

  0