Skip to content
OverviewSolutions
46
Comments
3
CairoGarb
@CairoGarb

All comments

  • CairoGarb•810
    @CairoGarb

    It looks great! The responsiveness works well. About the code, you can remove the repetitiveness in the "@media screen". You can set all the changes you want in only one (if they are the same).

    EX: @media screen and (min-width: 300px) and (max-width: 780px) { // major changes }

    Marked as helpful
  • CairoGarb•810
    @CairoGarb

    Well done! For the img, change the url to "images/avatar-jessica.jpeg" or "/images/avatar-jessica.jpeg". The 'images' folder is already inside the 'project' folder, so changing this would work.

    Marked as helpful
  • CairoGarb•810
    @CairoGarb

    Hey! Regardings of the .bg, maybe using display: flex. The measures "rem" or "pixels" instead of "%" could improve it. Flex would improve the alignment of the division, not needing to manually center the main container on the center of the page. I noticed that you used "position: absolute" on the image, maybe if you change the width of img to 100%, remove the absolute position and the margin left and in the "h2", remove the margin top aswell. Was well done, good job!

    Marked as helpful