Skip to content
OverviewSolutions
10
Comments
10
Tarek514
@Tarek514

All comments

  • Tarek514•170
    @Tarek514Posted about 2 months ago

    Add the hover effect to the text and image container

    Marked as helpful
    0
  • Tarek514•170
    @Tarek514Posted about 2 months ago

    Make the border-radius for the share button 50%

    0
  • Tarek514•170
    @Tarek514Posted about 2 months ago

    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

    0
  • Nostromito•190
    @NostromitoSubmitted about 2 months ago
    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.

    Nostromitos Testimonials grid section

    1
    Tarek514•170
    @Tarek514Posted about 2 months ago

    Nice job

    0
  • Tarek514•170
    @Tarek514Posted 2 months ago

    Think you used the repository link for the screenshot part instead of the live site URL

    0
  • Perry_Y_Z•130
    @Perry2004Submitted 3 months ago
    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.

    CSS Flexbox

    1
    Tarek514•170
    @Tarek514Posted 3 months ago

    Should lower the padding

    0
  • Tarek514•170
    @Tarek514Posted 3 months ago

    Looks good

    Marked as helpful
    0
  • Tarek514•170
    @Tarek514Posted 3 months ago

    Looks good, I would add height: auto to the container

    Marked as helpful
    0
  • Tarek514•170
    @Tarek514Posted 3 months ago

    Nice job creating the design, make sure to put the box-shadow and the border-radius for the image

    0
  • Tarek514•170
    @Tarek514Posted 4 months ago

    The text seems to be off in the solution compared to the design

    0