Community Feedback

• 2

  Hey Pierre, looks good! Just a few things here:

  • The component is touching the left and right gutter in medium widths up until the breakpoint

  • Icons could use cursor: pointer that suggests they are clickable

  • I would limit the use of ids for JS targeting or scroll links. They have a high specificity, meaning their styles are hard to override. Better to use classes.

  Hope this helps!

  P

  emestabillo3,860Posted 6 months ago

  Reply

  • 1

    @emestabillo Thank you for your comments ! I'll take a look into this and make some changes!

    

    pchatard215Posted 6 months ago

    Reply

  Add a comment
• 1

  Hey, nice work, Pierre! 👍

  Your solution looks good and is responsive! 👏

  I suggest,

  • Allowing the background SVG to fill the entire area of the screen in the mobile layout of the site. Currently, some white space begins to appear at the top and bottom of the viewport at very narrow screen widths.
  • Perhaps decreasing the right-padding in the box with the option icons just a little so that all of the icons remain on one row (even right before the layout changes from desktop to mobile).

  Keep coding (and happy coding, too)! 😁

  P

  ApplePieGiraffe19,700Posted 6 months ago

  Reply

  Add a comment

Give some feedback to pchatard about their solution