@prantiknoor
Posted
Your code is very well.😊 I learned some new things from your code.
But there is something that can be better.
Firstly, instead of using px
, you can use rem
. (rem
is relative to :root
font-size. Normally, 1rem == 16px
.
.description{
display: flex;
flex-direction: column;
/* gap:16px;
padding-bottom: 24px; */
gap: 1rem; /* 1 * 16 = 16px */
padding-bottom: 1.5rem /* 1.5 * 16 = 24px */
}
You can imagine 1rem
is letter's width or height. If you want padding as 1 letter you can use 1rem
, for 2 letter , 2rem
.
Secondly, The image hover effect is quite fast. If that could be a little bit lower & cursor: pointer;
then it would be more awesome.🥰
Marked as helpful
@prantiknoor Yes, I know that using other units instead pixels can be better, and working with rem is on my list of next things to do, but I only started learning at the start of July, so I still feel more comfortable with pixels for now (even if it's not the best). Thank you for your feedback! I will look to improve :)