Used HTML and CSS, learned about flexbot and other things

Community feedback

• 

  PhoenixDev22•16,950

  @PhoenixDev22Posted almost 3 years ago

  Hello @sirargill,

  I have some suggestions regarding your solution:

  • The images don't need filled alt . They are decorative images, so alt attribute should be left intentionally blank. You can optionally add aria-hidden or role presentation to ensure the images are always ignored by screen readers ANDavatar's one and image-equilibrium.jpgshould not be empty and should be descriptive . Read more about decorative and informative images .

  • You can simply use unordered list <ul> to wrap class="side-align’ and in each list item would have <img > and <p> `.

  • You don’t need <hr > , you can use border-top for the class="side-align .

  • You can use more html semantic tags like <figure> <figcaption > for the avatar part.

  • Add the hover effect on the image . (The eye image is missing)

  Hopefully this feedback helps.

  Marked as helpful

  0

  ---

  

  Ujjwal Singh•70

  @sirargillPosted almost 3 years ago

  @PhoenixDev22 thank you for your kind suggestions, I will make sure to implement those.

  0
• 

  sb101-beep•60

  @sb101-beepPosted almost 3 years ago

  Try styling .container with flexbox centering.

  0

  ---

  

  Ujjwal Singh•70

  @sirargillPosted almost 3 years ago

  @sb101-beep Thank you for your suggestion.

  0
• 

  Daniel Hernandez•60

  @jhernarPosted almost 3 years ago

  you can try giving a min-height of 100vh in the body section

  0

  ---

  

  Ujjwal Singh•70

  @sirargillPosted almost 3 years ago

  @jhernar Thank You for the kind suggestion.

  0