Skip to content
OverviewSolutions
25
Comments
3
Faith
@fnwork

All comments

  • Faith•420
    @fnwork

    Hello, here are some suggestions

    -Your alt tag should have a description in it like "Victor" or "profile picture for Victor" checkout: Checklist - The A11Y Project #for-images-containing-text-make-sure-the-alt-description-includes-the-images-text

    -The two circles should have been included in the HTML file (that's how I did mine) and you use position-absolute to position the circle in their respective directions checkout: https://www.w3schools.com/css/css_positioning.asp

    Good luck on your next project and happy coding

    Marked as helpful
  • Faith•420
    @fnwork

    Hello, I want to share with you some of the things I have learned here so far from amazing people that you can use in your next project.

    1. Your font size should be in rem or em and not px
    2. it is good to move your CSS style to another file, then link it to the HTML file
    3. make sure to take out the commented sentences before you deploy your project
    4. if you have some time, please read on Flex and Grid - it will help you with the layout
    5. lastly, I was introduced to https://utopia.fyi/ (helps with layout, font size, etc.) and https://www.a11yproject.com/ ( is a checklist that helps you with accessibility on your site) - check it out

    Good luck with your next project

  • Faith•420
    @fnwork

    Hey there, I just checked your solution and you did good but there is room for improvement. Please here are some links I got from great people over here that will help with your next project.

    1. https://www.w3schools.com/css/css3_mediaqueries.asp - this link will help you learn about media queries in order to achieve a responsive website
    2. https://www.w3schools.com/tags/tag_picture.asp - this will help you with displaying the appropriate picture for the screen size

    I wish you all the best and happy coding

    Marked as helpful