@ApplePieGiraffe
Posted
Hey, nice work on this challenge, Oleh Tovkaniuk! π
Your solution looks good and works rather well! π
The only thing I might suggest is perhaps preloading the background images so that there is no delay between the light/dark theme button changer is pressed for the first time and the image appears. It would be a nice touch, I think!
Also, I'm unable to check off items on the to-do list as completed for some reason. You might want to look into that to see what's going on! π
Keep coding (and happy coding, too)! π
@OlehTovkaniuk
Posted
@ApplePieGiraffe Good points, thanks!
You can check off if you press at a todo's name. But if it confuses a user then it is a bad design and should be reviewed.
I believe that i should add an event listener for that visual checkbox element too.
@ApplePieGiraffe
Posted
@OlehTovkaniuk
Oh, I see. Yes, I think it would be a good idea to allow users to mark items as completed by clicking on the checkbox as well as the name of each todo. Otherwise, I tried clicking on the names of the todos, and it works as expected! π
@OlehTovkaniuk
Posted
@ApplePieGiraffe Thanks a lot, my friend. Could you pls also have a look at my JavaScript code structure. Could you give me few advices how to make it better?
@ApplePieGiraffe
Posted
@OlehTovkaniuk
I'm not super good at JS or anythingβbut you're JS looks fine to me. It's nice that you've separated your code into lots of functions (which helps keep your code DRY and therefore easier to maintain and stuff) and the comments make it a lot easier to understand what's going on at a glance. π
I don't know if you've already seen this, but a good example of a to-do list made in vanilla JS is this awesome video from Dev Ed. If you like, you can take a look at that and perhaps learn something new! π
@OlehTovkaniuk
Posted
@ApplePieGiraffe you won't believe it but I actually seen this video, moreover my function deleteCheck is from this video and this is a way of solving two features - delete and check of a todo at once. It is a super intelligent idea, isn't it
@ApplePieGiraffe
Posted
@OlehTovkaniuk
Haha, I actually followed the entire tutorial from start to finish as a sideproject! π You're rightβDev Ed is really smart and his videos are always so informative and funny! π