Skip to content
OverviewSolutions
10
Comments
3
Rahaf
@rahafsz

All comments

  • Rahaf•170
    @rahafszPosted about 2 months ago

    Nice work! I have a note: every element has hovered which means it is clicked so instead of using span in the footer use an anchor tag and for img tag use an anchor to include the image. PS: When making a card on a web page we don't use the h1 tag for the title we use another one of the heading level.

    0
  • Rahaf•170
    @rahafszPosted about 2 months ago

    Congratulations on finishing the project! I have a note, the webpage has a vertical scroll that comes from the width of the body 100vw. Great job at using fonts! I was looking for this way and finally found it. I will use it in my next projects.

    0
  • PreANI•50
    @preani19Submitted about 2 months ago
    What are you most proud of, and what would you do differently next time?

    the similarity between the result and the design

    What challenges did you encounter, and how did you overcome them?

    mobile design, by using CSS units

    social links profile main

    1
    Rahaf•170
    @rahafszPosted about 2 months ago

    Congratulations on the finished project. I just noticed that small issue you made the container position absolute without position relative to the parent element, and you made the container the middle of the web page by using position the way it works but it isn't the best way for this condition instead of it use for body {height: 100vh; display: flex; justify-content: center; align-items: center;}

    Marked as helpful
    1