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

  • SAKIB9922 10

    @Sakib9922

    Submitted

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

    I'm proud that I could complete the project without any problems

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

    I did forget to add something like hover effect but good thing I checked others solutions before submitting. also I use w3schools if I forget any code

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

    I'm not sure if I add any unnecessary elements also I add some comments so its easier to understand not sure if it's right way or not

    @cmdaltent

    Posted

    Looks good to me! 👍 The solution is really close to actual design. Very good job.

    As an improvement, I noticed you put margins on many items that are children of the card_content in order to get the desired spacing. Since spacing among the items in the container is the same in the design, I would suggest using the gap property in the card_content to give all flex items the same spacing. That way, the individual items do not need to care about their spacing within their container, which would potentially increase reusability.

    0
  • @cmdaltent

    Posted

    Solution looks good to me. 👍 Keep up the good work!

    I just noticed that the background color differs from the design. It should've been the slate 300 color – same for the text in the p tag. That also leads to the --Slate-300 variable to be unused in your code, which is generally not recommended to keep unused variables in your code base as it leads to clutter and dead code.

    0