@sztedina
Tarek514
@Tarek514All comments
- @Tarek514
Should change the background color of the page and reduce spacing of the bullet points.
- @BlasVernazza06@Tarek514
Make the border-radius for the share button 50%
- @Tarek514@Tarek514
For some reason, the screenshot shows the text outside of the card; but on the website, the text is inside the card like the design
- @NostromitoWhat 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.
@Tarek514Nice job
- @mohamedsaad2006@Tarek514
Think you used the repository link for the screenshot part instead of the live site URL
- @Perry2004What 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.@Tarek514Should lower the padding
- @reemoon-ctrl
- @karthika200130
- @rhuann77@Tarek514
Nice job creating the design, make sure to put the box-shadow and the border-radius for the image
- @esdrasmdz@Tarek514
The text seems to be off in the solution compared to the design
