Skip to content
OverviewSolutions
50
Comments
4

Shady Omar

@Shady-OmarAlexandria950 points

a Junior-Level Front-End Developer.

I’m currently learning...

Next.js

Latest solutions

View more solutions

Latest comments

  • Shady Omar•950
    @Shady-OmarPosted over 2 years ago

    Hello, Good job on completing this challenge !!

    My advice here is to use display: flex; to horizontally center your avatar instead of using margin property for better accuracy as you see the image it not well centered.

    Other than that all seems good, Hope this helps

    keep going, Enjoy coding !!

    1
  • Shady Omar•950
    @Shady-OmarPosted over 2 years ago

    Hello gcWDev,

    Good job on this challenge, But here is a tip to show the images of this design while using GitHub pages

    = change the src attribute of the img tag from src="/images/image-equilibrium.jpg" to src="images/image-equilibrium.jpg"

    = Also the change avatar image src attribute from src="/images/image-avatar.png" to src="images/image-avatar.png"

    Hope this helps.

    Marked as helpful
    0
  • Shady Omar•950
    @Shady-OmarPosted over 2 years ago

    Hello Kevin,

    Nice work on this challenge, But here is a tip to show your QR code image while using GitHub pages

    = change the src attribute of the img tag to src="images/image-qr-code.png" instead of the src="../images/image-qr-code.png"

    Hope this helps.

    Marked as helpful
    0
  • Shady Omar•950
    @Shady-OmarPosted over 3 years ago

    The result is great, But as for your HTML, I think you should use <div> tag more often instead of using <section> tag,

    <section> tag is more used as to include all <div> tags in it, but not include sections inside sections,

    I hope this helps.

    1