Community feedback

• 

  MikevPeeren•2,100

  @MikevPeeren

  Posted over 3 years ago

  Hey 👋

  I can't really find anything other than maybe the image overlay being of a different color. Other than that great job 👏

  11

  Marked as helpful
• 

  Raymart Pamplona•16,040

  @pikapikamart

  Posted over 3 years ago

  Hey, great work on this one. Desktop layout looks great, just the overlay is bit too bright. Resizing the screen creates issues on the site. If you go to like 550px, you will notice that the image on the right side is now very small and just looking at 470px upwards, the image is not being resized or doesn't scale that well. Though the mobile state looks fine.

  Some suggestions on the site would be:

  • Change the header into using main tag since a primary header usually contains the topmost part of the site which includes the website-logo, the navlinks and some other controls. But for this, there are none and this whole layout is the main-content of the site itself.
  • br on the h1 is not needed since it will automatically wrapped the text on another row since the container will limit the h1 size.
  • You could used ul for the .lower selector since those are list of items that are inside it.
  • When wrapping up a text-content, make sure that it is inside a meaningful element like p tag or heading tag and not using like div, span to wrap the text. Use p tag on the texts like companies and other.
  • Add an extra aria-hidden="true" on the image so that it won't be picked up by other screen-reader. Though on this one, I find the image meaningful and I would add a descriptive alt on it.
  • Remove the position: absolute on the footer tag since it just gets on the screen when using dev tools at the bottom.
  • Lastly, addressing the issue about the responsiveness especially for the image.

  Aside from those, great job again on this one.

  01

  Marked as helpful