Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • @vladzen13

    Submitted

    What are you most proud of, and what would you do differently next time?

    Nice responsiveness till ~200px, no media queries or harcodes. Img width with % seems like nice idea

    What specific areas of your project would you like help with?

    Vertical+horizontal center with flex still shrinks div.profile a little bit, need deeper understanding of flexbox

    @Ihdir

    Posted

    Hey there,

    Nice work on the project! I noticed a few things that might help improve it:

    1--It's best practice to include a <main> tag inside your <body> to highlight the primary content.

    2--Applying a full modern CSS reset can simplify things as you build. You can check out this site for a full reset.

    3--When it comes to responsiveness, try using max-width: 100% or min-width: 100% instead of just width: 100%—they work better in responsive designs.

    4--Avoid using px for units since it's absolute. Consider using rem or em for more responsive layouts.

    5--Check this article for deeper understanding of flexbox: https://developer.mozilla.org/en-US/docs/Learn/CSS/CSS_layout/Flexbox

    Hope you find these tips helpful!

    Happy coding! 😄

    0
  • @Ihdir

    Posted

    Hey there!

    It looks like your GitHub repository link isn't working properly. Make sure to fix that, and double-check the image file path as well.

    Happy coding! 😄

    0