@elaineleung
Posted
Hi Anderson, great job here, and I think everything looks great (looking great is always the first step to success)! As for functionality, I was able to add items and remove them using the "X", but the filter options below weren't really working for me, as in they don't seem to be responding.
Anyway, one suggestion I have give here is the background image; I'm viewing this on a large screen and the image is cut off on the right. To prevent that from happening, just add background-size: contain
and that should fix it.
About a color theme, I recently had to build one as well, and I found Adam Argyle's tutorial here to be a great help: https://web.dev/building-a-theme-switch-component/
Here's the challenge I built that used the theme switcher: https://www.frontendmentor.io/solutions/responsive-calculator-built-with-vanilla-js-w9UinqToJQ
Anyway, this is a great start, Anderson!
Marked as helpful
@andykallian
Posted
@elaineleung Hi Elaine
thanks for your suggestion about background-size. I've implemented it in project and as soon as possible going to push it to github repository.
Also, I'm going to check your links here. I think they will be such a great help! Thank you!!
@elaineleung
Posted
@andykallian I just checked and the background looks great! 🙂