Responsive Social links profile using Vite and SCSS

Community feedback

• 

  P

  Steven Stroud•11,910

  @StroudyPosted about 2 months ago

  Exceptional work! You’re showing great skill here. I’ve got a couple of minor suggestions that could make this stand out even more…

  • I would not put your list inside a nav, This is not semantically correct, Nav is used for navigation on the site from page to page not external links,

  • This does not matter that much at this stage but something to be mindful of for SEO(Search Engine Optimisation), <meta> description tag missing that helps search engine determine what the page is about, Something like this <meta name="description" content="description goes here" />

  • Hidden tip here, Mixins are coming to native CSS, SCSS will become less popular as more and more features come to native CSS.

  You’re doing so well, and I hope this feedback is helpful! Keep honing your skills and remember to enjoy the process—you’re on a great path. Stay motivated, and happy coding! 🎯

  Marked as helpful

  0

  ---

  

  P

  Saumyadip Biswas•50

  @biswasSaumyadipPosted about 2 months ago

  @Stroudy thank you for your suggestions. Will follow these from now on.

  1

  

  P

  Steven Stroud•11,910

  @StroudyPosted about 2 months ago

  Hey @biswasSaumyadip, You are smashing it bro, I struggled to give you any of my limited feedback.

  0
• 

  Anderson Moreira Silva•70

  @anderjackiePosted about 2 months ago

  Your solution looks amazing. Almost perfect with the design. Very good. It is a nitpick but I would separate the reset css rules from the design style in 2 separate files. I believe it is a good separation of concerns.

  1

  ---

  

  P

  Saumyadip Biswas•50

  @biswasSaumyadipPosted about 2 months ago

  @anderjackie yes I should have done that. Thank you.

  0