@rostyslav-nazarenko
Posted
Hello! Nicely done!
- HTML
- use
main
andfooter
for semantics. Change<div class="section">
to<main class="section">
, and<div class="attribution">
to<footer class="attribution">
. - look at the
picture
element for providing different images to the different viewport widths, plus it has a lot more uses, check it up. - don't use
id
for styling, use only classes. - look at providing additional information for people who use screen readers as they will not understand what two prices mean. I'm still learning about accessibility...
- use
Your CSS has some issues, keep learning and you will notice them yourself. Feel free to look at other people's solutions for learning purposes.
Keep coding! Have a great day!
Marked as helpful
@Udy-Johnson10
Posted
@rostyslav-nazarenko thank you for the feedback