Latest comments
- @TremerYT@komaljatoi
Looks good! Two quick comments: (1) on your solution, the picture of the perfume doesn't seem to be loading; (2) I would suggest the color hsl(30, 38%, 92%) as the background to be closer to the challenge design.
- @RickLearnsToCodeWhat are you most proud of, and what would you do differently next time?
I feel pretty good about how my 2nd challenge went. There were a few issues with spacing that I was able to figure out without the help of Copilot, and this was good practice with flexbox. I'm still not sure why my fonts don't look like the provided design image, and I am not sure how to adjust the font sizing for smaller screens without using media queries. Looking forward to seeing other solutions to this one.
What challenges did you encounter, and how did you overcome them?I downloaded Figma so I could view the provided files, but really struggled to figure out how to use that tool. Ultimately I just used the .png files as my guide, but I need to spend a little time getting up to speed on Figma for future challenges.
I also initially struggled with flexbox to get the card elements aligned vertically until I realized I needed to use flex-direction: column. (Facepalm moment but a good learning experience)
What specific areas of your project would you like help with?The width of my card (or possibly my font sizes?) are not matching the example, so my excerpt text only takes up 2 lines instead of 3. I also notice the font weight of my title and the author name don't seem to be as bold/thick as the example. Not sure what I need to do there to fix it.
@komaljatoiLooks really good! One suggestion you could try for the font weight - instead of font-weight: bold, you could try font-weight: 800, which would make the title/author bolder.
Marked as helpful - @onkarnikWhat are you most proud of, and what would you do differently next time?
learn box-shadow
@komaljatoiThis looks great! To get it closer to the sample design, I would suggest rounding the corners around the yellow box with "Learning" using border-radius!
- @DuyTM0508@komaljatoi
This looks terrific! To change the color of just the numbers in the ordered list, you could try using --- ol li::marker ----.
Marked as helpful - @DuyTM0508@komaljatoi
Looks great! I noticed that your CSS code is within your HTML; you could consider coding the CSS in a separate file like styles.css if you like that approach.
Marked as helpful - @Shakiba-Alipour@komaljatoi
Good work! I would just recommend using CSS properties to center the blog preview both vertically and horizontally on the page!