Blas Vernazza• 110
@BlasVernazza06
Submitted
Tarek514
@Tarek514
All comments
- Tarek514• 130
@Tarek514
Posted
Make the border-radius for the share button 50%
0 - Nostromito• 190
@Nostromito
Submitted
What are you most proud of, and what would you do differently next time?
Getting a better understanding of grid is what I'm most proud of and I will continue to learn about its usefulness.
What challenges did you encounter, and how did you overcome them?
My biggest challenge was getting the equations to stick in place and getting grid to work how I needed it to. I am still looking outwards for help but I can see myself getting better with everything I already have a minor understanding of.
What specific areas of your project would you like help with?
Any constructive criticism is welcome.
Tarek514• 130@Tarek514
Posted
Nice job
0 - mohmed saad• 60
@mohamedsaad2006
Submitted
Tarek514• 130@Tarek514
Posted
Think you used the repository link for the screenshot part instead of the live site URL
0 - Perry_Y_Z• 130
@Perry2004
Submitted
What specific areas of your project would you like help with?
It's a bit weird that when the viewport is stretched to very big, the left image won't cover the entire div although I've set
object-fit: cover. I've checked multiple source still didn't figured it out.Tarek514• 130@Tarek514
Posted
Should lower the padding
0 - karthika200130• 30
@karthika200130
Submitted
Tarek514• 130@Tarek514
Posted
Looks good, I would add height: auto to the container
Marked as helpful
0
