Meet landing project

Community feedback

• 

  P

  Andrew A Lee•380

  @drewlee

  Posted 4 months ago

  The page is functional and overall fairly close to the spec. I'd suggest a few improvements to more accurately match the visual design.

  • The black heading color is harsh and should be set to the "Slate 900" variant that is established in the style guide.
  • Same issue for the section markers (e.g., "01", "02"), which should be the "Slate 600" variant.
  • The main heading should be a smaller font size for mobile screens.
  • The user pictures in the grid should feature a border-radius.
  • The footer background color should be "Cyan 600" instead of black.

  Additionally, I would look into leveraging more semantic HTML elements for your markup, such as heading, main, footer, section, and article elements. Divs should just basically be reserved for stylistic only purposes or used as a last resort when there are no better options.

  00