Responsive Social profile page using flex box

Community feedback

• 

  Shoaib Bin Rashid•120

  @Shoaib-Bin-RashidPosted 3 months ago

  great

  Marked as helpful

  0
• 

  Affan Amer•370

  @AffanRMPosted 3 months ago

  Hey, upon viewing your website I have a few suggestions for you to improve your code and design:

  • Instead of using <button> to specify the navigation links, it is better practice to use anchor tags (<a/>) as anchor tags should always be used when presenting a link to another website.
  • Instead of specifying the font-size in pixels (px), it is better to specify it in rem/em units so that your website is more accessible for all users.
  • Your website is not vertically aligned on my laptop screen, one way to achieve this is by wrapping your box class in another <div class='x'> and then setting the following properties for this <div>

  .x{
    min-height: 100vh;
    display: flex;
    align-items: center;
    justify-content: center;
  }
  If you need any further help, then let me know!
  

  Marked as helpful

  0

  ---

  

  Adebukola Adeyinka Omowole•280

  @AdebukolPosted 3 months ago

  @AffanRM thanks for this awesome review, I will implement it.

  0