@emestabillo
Posted
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
id
s for JS targeting or scroll links. They have a high specificity, meaning their styles are hard to override. Better to useclass
es.
Hope this helps!
@pchatard
Posted
@emestabillo Thank you for your comments ! I'll take a look into this and make some changes!