Used HTML and CSS, learned about flexbot and other things

Community feedback

• 

  PhoenixDev22•16,830

  @PhoenixDev22

  Posted over 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.

  01

  Marked as helpful
• 

  sb101-beep•60

  @sb101-beep

  Posted over 3 years ago

  Try styling .container with flexbox centering.

  01
• 

  Daniel Hernandez•60

  @jhernar

  Posted over 3 years ago

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

  01