All comments

• 

  SAKIB9922•10

  @Sakib9922Submitted 4 months ago

  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

  

  Blog preview card using flexbox and variables

  1

  20
  

  Martin Weissbach•20

  @cmdaltentPosted 4 months ago

  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

  This comment has received 1 replyShow
• 

  Volkan Şener•40

  @VolkanSenerrSubmitted 4 months ago

  

  qr code challange solution

  2

  10
  

  Martin Weissbach•20

  @cmdaltentPosted 4 months ago

  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