TCO932• 230
@TCO932
Submitted
Rahaf
@rahafsz
All comments
- Rahaf• 170
@rahafsz
Posted
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 - Veera Bramha Chary• 150
@Veerabramhachary
Submitted
Rahaf• 170@rahafsz
Posted
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
@preani19
Submitted
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
Rahaf• 170@rahafsz
Posted
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
