abderrahmane bnd• 150
@abderrahmanebnd
Posted
Salam, I liked how you used the semantic tags but i have some observations: 1- What happened if you will decide that color hsl(47, 88%, 63%) have to change to blue for example? so your variable --yellow: will be blue. And that is a mess. use something neutral for that, for example:
:root {
--primary-color: hsl(47, 88%, 63%);
}
2 - add
.review-card {
cursor:pointer
}
to know that this is clickable, maybe u can add transition also .
3- about your problem : just try to estimate the spaces between elements .
1
amina-refik• 80
@amina-refik
Posted
salam @abderrahmanebnd ,
Thank you for your feedback. I've incorporated your suggestions, and I will explore further on how to incorporate CSS transitions.
0
abderrahmane bnd• 150
@abderrahmanebnd
Posted
you can also do it with the other colors I mean the yellow one was just an exemple ,
--color-brand-1:white,
--color-branb-2:red ,
and so on ,
0
