@Alex-Archer-I
Posted
Hi!
The short answers for your questions are yes and no =)
Yes, you should use semantic tag. At first it's a main
tag - it could be .card
element in your case. Than you could replace all the span
with p
(actually "Learning" thing looks like the link, but this project is out of context, so never mind =)).
And no, you shouldn't wrap every element to div
, it's just create more unnecessary tags. Your layout is quite good now =)
The clamp()
function takes three parameters. The first and last are defines the minimum and maximum of the value. And the middle one is relative value which would change within that range. In your case the font size will be 1.1vw
(which means 1,1% from width of the screen) but would never be smaller than 12px
and larger than 14px
.
And responsive tips. Try not to hardcore width
for text elements (I mean .tag-learning
and .date
) and height
for the container element. In real project the content could change time to time, so it's better if the space would adjusting to it.
Hope that could help. Good luck in your frontend journey and fell free to ask anything if need =)
Marked as helpful
@Arnotts33
Posted
Hi @Alex-Archer-I !
Thanks for the very helpful and thorough feedback.
@Alex-Archer-I
Posted
@Arnotts33
You're welcome =)