Social links card

Community feedback

• 

  Alex•3,130

  @Alex-Archer-IPosted 3 months ago

  Hi!

  Very cool work, especially considering time you spend =)

  One semantic suggestion - for the lists better use ul or ol tags. And I guess you accidently deleted opening main tag.

  And I like your animated hover! If I understand correctly, the pseudo element and position: relative for text are for this effect?

  Marked as helpful

  0

  ---

  

  aurelien•100

  @pichadePosted 3 months ago

  Hi @Alex-Archer-I !

  Thanks for the feedbacks, just fixed these small mistakes !

  And yes de position:relative is needed as I'm using absolute on the pseudo element.

  1

  

  Alex•3,130

  @Alex-Archer-IPosted 3 months ago

  @pichade

  Oh, now I noticed that you are from motion design field. Yeah, the animations should be your element =)

  And, sorry, but you get me slightly wrong =)

  You shouldn't remove the main - it is important semantic tag. And li (it's a "list item") should be inside ul (it is a "unordered list").

  Something like this:

  <ul>
      <li><a href="">I'm a link!</a></li>
      <li><a href="">I'm a link!</a></li>
      <li><a href="">I'm a link!</a></li>
      <li><a href="">I'm a link!</a></li>
      <li><a href="">I'm a link!</a></li>
  </ul>
  

  0

  

  aurelien•100

  @pichadePosted 3 months ago

  @Alex-Archer-I oh man ! I feel stupid ahah, i wanted to do it fast and didn't even noticed that I've put li's as parents of a's 🤦‍♂️

  1

  

  Alex•3,130

  @Alex-Archer-IPosted 3 months ago

  @pichade

  Haha, It's a backside of speedrun =)

  0
• 

  Lance•1,580

  @IzykGitPosted 3 months ago

  Now I have the urge to go back and speedrun this project myself.

  2

  ---

  

  Alex•3,130

  @Alex-Archer-IPosted 3 months ago

  @IzykGit

  If you already did it you can beat all records and speedrun it for couple of seconds by "copy-paste" =)

  1