Social Links Page with HTML, CSS and techniques like flexbox

Community feedback

• 

  Alex•3,130

  @Alex-Archer-IPosted 5 months ago

  Hi! Interesting work, it's always cool to experiment with design =)

  Here is a few tips:

  • Use min-height instead of height on the body tag in case if the user's devise have smaller screen than content.
  • Almost the same about .article - try not to explicitly set height for container elements. In real projects there could be the cases when content might changed, so it's better when height of a container depends on the content.
  • It is better use ul and li tags for the buttons-links because of semantic. To keep html leaner you could put a directly into li, buttons aren't really necessary here.

  Oh, and the padding property haven't auto value.

  Anyway, cool job, keep doing =)

  Marked as helpful

  0

  ---

  

  Prinz•60

  @maxnoelpPosted 5 months ago

  @Alex-Archer-I thanks for ur comment, i learn since 2 weeks. I will keep in my mind :)

  1

  

  Alex•3,130

  @Alex-Archer-IPosted 5 months ago

  @maxnoelp That's cool, good luck with your studying =) Fell free to ask or comment if you need =)

  1

  

  Prinz•60

  @maxnoelpPosted 5 months ago

  @Alex-Archer-I thanks, it is a lot to learn :) I may take you up on that later.

  1
• 

  Umesh Suwal•180

  @umeshsuwalPosted 5 months ago

  The font is little bit hard to read, it's not good for user experience

  0