@denielden
Posted
Hello Katja, You have done a good work! 😁
Some little tips to improve your code:
- add
main
tag and wrap the card for improve the Accessibility - use
p, strong, h1, h2, h3
tags instead of a simplediv
for the texts inside notification - remove all unnecessary code, the less you write the better as well as being clearer: for example the
picture
container of image...picture
tag is good to change image by resolution -> read here
Keep learning how to code with your amazing solutions to challenges.
Hope this help 😉 and Happy coding!
Marked as helpful
@katjadanilova
Posted
@denielden Thanks, Deniel, for your comments. I finally had time to get back to this small project, and implemented some of the changes you recommended, and improved accessibility - indeed, a very important step!
@denielden
Posted
@katjadanilova You are welcome and keep it up :)