@NorCoolTech
Latest comments
- @alepacc
Great work!
A small detail: the border color of the avatar image on card number four should be purple.
- @Sahithkumar-VU21CSEN0100489@alepacc
Great job!
- you didn't link the correct live page (but I found it on GitHub)
- in the firstpart you can use h1 or h2 header tag and the inline CSS code you can move to the external CSS file
- the text size in firstpart should be a bit bigger
- @ebrahim113@alepacc
Hello! Congrats, perfect job!
- @Katherine-Kapinus@alepacc
Hello! Congrats, perfect job!
- @mangoloving@alepacc
Hi! Your site look good, but you can improve a little things.
- you can use semantic html
- the text London, United Kingdom should have a different color — Green: hsl(75, 94%, 57%) (you can find in file style-guide.md)
- To add space between elements, you can also use the gap property with Flexbox
Marked as helpful - @ClavijooWhat are you most proud of, and what would you do differently next time?
I was able to find a solution to move the shadow by myself, and the transition when you hover over the card is cool
What challenges did you encounter, and how did you overcome them?It took me a little bit to find out what display option to use to align both the avatar and the text. I had to do my research
What specific areas of your project would you like help with?How can I improve my CSS code?
@alepacc- the border of the card in figma was 1px
- for mobile version, the different font size is missing
- for the padding in the card you can use:
display: flex; flex-direction: column; gap: 24px;
