@phamyuuto
Latest solutions
Blog preview (with multiple tags)
#bem#accessibilityPSubmitted 4 months agoI don't know - it seemed easy.
QR Code
PSubmitted 4 months agoI struggle to understand where use px, em and rem and about accessibility.
Latest comments
- P@ilierette
Looks good! Good job with using custom variables in css.
I noticed is lack of borders of card component. It have shadow, but also borders. That's a minor style issue.
Also I might be wrong, but I don't know if "article" is good semantic tag for it - it's more like card component than article. I would use main and put inside div with class card.
Marked as helpful - @AbanobRafikP@ilierette
Hello! First feedback there, so take it with a grain of salt. Overall looks good and very similar to project, good job! But was the background choice deliberate? It's grey on design. And divs are not semantic tag. That's all, keep up!
