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

  • P

    @DarrickFauvel

    Submitted

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

    I like using custom CSS properties and integrating them into Tailwind CSS, but I think next project I will put my custom color names and values directly into the Tailwind config file.

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

    No real challenges with this project, but I look forward to the next.

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

    I am open to any constructive feedback that can help me improve as a developer. 😊

    @code-elias

    Posted

    Very well written code. Love the hover effects on the image as well.

    The solution is basically identically to the design.

    1
  • rshnsgh 20

    @rshnsgh

    Submitted

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

    I'm most proud of my ability to sort and organize everything given that this is my first time attempting a project of such.

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

    The two main challenges i encountered was the nutrition table and the mobile vie that changes with size. I overcame both by a simple google search that showed me how close I was to getting it right without having to do research.

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

    I would say mainly the CSS

    @code-elias

    Posted

    Hi! The solution looks pretty close to the design. I'd only suggest extending the body background all the way to the bottom and make the font-weight a little thinner.

    0
  • @code-elias

    Posted

    A well-written and navigable Vue app with a good separation of components and assets. I personally really like how you separated the data into its own file, I think I'll start doing the same, everything looks a lot neater. It adapts to various screen sizes quite well .

    A minor suggestion I have is to make the hover effect on the light theme just a bit more prominent, at the moment the effect is difficult to notice. But that's more of a personal preference because I like interactivity.

    Thank you for the Amazing link haha

    0
  • @code-elias

    Posted

    Hi! Very clean HTML structure and great use of SCSS. I'd suggest adding some spacing in the body of the card to match the design better, but it's a very minor nitpick.

    0
  • @ChrisThompsonDev

    Submitted

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

    This project was very simple. I am proud that I was able to dust off my CSS and easily build this project. I would like to incorporate responsive font sizing next time using Font Clamp.

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

    I had trouble with selecting the colors appropriately. My monitors did not have the same color settings and it made it difficult to discern the light blues and grays.

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

    N/A

    @code-elias

    Posted

    Hi, the solution looks pretty awesome. The height and the padding seems just little different from the design, but that's a nitpick.

    0
  • @code-elias

    Posted

    Hi! Honestly I'm impressed by the quality of your code. It's very clean and organized and love the use of the the data-icon to add the shopping cart element and the picture tag (I need to use it more often).

    The only minor suggestion I would give is adding a max-width of 400px to your product card for smaller screens, and then move the max-width of 600px in the media query.

    But otherwise, honestly, It's basically pixel perfect!

    Great Job and keep it up!

    Marked as helpful

    1
  • @code-elias

    Posted

    Hi Adriel, Your solution is a bit different from the original design, but I don't think it looks bad at all. I would only suggest to create some additional spacing between the various elements. In particular the "Annual Plan" from the buttons below.

    In terms of accessibility, I think always adding an alt to your images is just a good habit to get into. I know this is just a small challenge, but it's always good to put it. Even something simple like "Dancing person".

    I don't know if this design showed a mobile version, but just letting you know that the card compresses very strangely on smaller screens. Responsive design is a pretty big argument, but some starter tips are generally to avoid vw for width, and instead using a combination of margins and max-width to avoid strange stretching and compression of your elements.

    Hope this is helpful, otherwise great job and keep it up!

    Marked as helpful

    0