Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • @adewalemudasiru

    Submitted

    What are you most proud of, and what would you do differently next time?

    This challenge took longer than expected but It feels good I was able to complete it. I decided on some implementation which I think is not the best approach to my challenges, but It work😂. I'll probably do it better once I figure out a better design approach.

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

    My biggest challenge during this project was placing the content card/box in the correct position. I didn't immediately think the CSS grid was the solution, so it took me longer trying to figure out how to position the boxes. I just started learning Grid actually and I guess I didn't understand I could use Grid in such a manner. I stumbled upon a similar design and inspected the code and saw what they did, so I just copied their code to Codepen and played with it, it seems easy but I still feel I don't understand it enough but hey, it worked😂. I also had a hard time getting the title description displayed correctly. I had to set a min-max for the description to get the result I wanted but I'm not sure if that was the correct solution but again it worked 😁.

    What specific areas of your project would you like help with?

    I would love to get more clarity on the CSS grid. While ChatGPT and Claude Ai have been helpful, I think getting a professional to help me understand the concept would help me improve my skills a lot.

    @Yahia-kilany

    Posted

    Good work. Using clamp for sizes is a good practice from what I hear though I am not sure of the ::before tags when you could just do border-top:5px; border-color:... It's way neater than my solution tho lmao.

    0
  • @DarkCityTreno

    Submitted

    What specific areas of your project would you like help with?

    Just wanna keep getting better at it even if slowly.

    @Yahia-kilany

    Posted

    The code is nice, using CSS variables and responsive design effectively imo. However, you should consider using the letter-spacing property on "perfume" to achieve the same effect as in the original design. it should be something like this

    .product-type{
    letter-spacing: 3px;
    }
    

    other than that I don't think I can give any other advice.

    Marked as helpful

    1
  • @Yahia-kilany

    Posted

    pretty clean and solid code in my opinion...I can't see the issue with it. However, the font weights and the color of the divisor lines aren't very accurate to the original which is a bit trivial tbh. Bravo!!

    0
  • arun81302 60

    @arun81302

    Submitted

    What are you most proud of, and what would you do differently next time?

    first time i use media query

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

    actually i am currently practising flexbox and media query

    What specific areas of your project would you like help with?

    It is good

    @Yahia-kilany

    Posted

    The design is nice but completely different from the original.

    0
  • @Yahia-kilany

    Posted

    I think you should put the display: flex;, justify-content: center;, and align-items: center; properties in the body element instead of the container class. The container div does not include the entire page so it can't really center the card vertically. otherwise, I think this is solid.

    0
  • @Yahia-kilany

    Posted

    neat solution

    0