Vee• 240
@JunasVee
Submitted
I would say that by far this is the easiest challenge on this platform.
If I missed something, please let me know.
Manigandan E
@iamurmani
All comments
- Manigandan E• 200
@iamurmani
Posted
Hi, Your code looks good, I have few observations
- The car icon is missing for luxury card as you misplaced your linkedin profile link. 2.Use the background color on body tag not on the main tag. If its used on main tag as the screen resolution reduces the background will be looking partial white & background color.
Happy coding!
0 - Akash• 80
@Akash20x
Submitted
hello frontend Mentors, what do you have for me about this solution? Please do not forget to comment and like :) really appreciate.
Manigandan E• 200@iamurmani
Posted
Hi Akash, Your code looks super. i have one observations, the social media icon circle is not uniform, to make the circle perfect you can use set a equal width and height and use flex box to align the icon to the center of circle for social div. Something like this
border: 1px solid #fff; border-radius: 50%; justify-content: center; align-items: center; height: 2.5rem; width: 2.5rem; font-size: 1.2rem; color: #fff; Happy Coding!!Marked as helpful
0 - Mithisar Brahma• 50
@moznony
Submitted
I'm a noob, looking for suggestions
Manigandan E• 200@iamurmani
Posted
Hi Your code looks awesome I have 2 observation
- On hovering on the social media icon as per design the border should also highlight. You can remove the ion-icon:hover and combine border & icon as given below .m-icon:hover { color : var(--Soft-Magenta); border-color : var(--Soft-Magenta); }
- I feel the heading h1 tag font size is big for mobile screen, if we reduce, it will look good.
Happy coding.
Marked as helpful
1
