@Sagameister
Posted
Looks, fantastic! The only tiny improvement I can see is in the HTML mark up you can make it more accessible like so:
- Consider rewrite <main> to <main class="card"></main> and remove <div class="card">. You also save 2 lines. ;-)
- Consider changing <div class="text-container"> to <article class="text-container">...
By the way, I love the way you used the CSS variables. I need to do more of that.
Cheers. 😃
Marked as helpful
@GSD3v08
Posted
@Sagameister I changed the lines ,thank you very much for your advice that helps me alot to improve my code.