Why you didn't use progress or meter for progress bar?
why make everything need a class rather than just styling the DOM elements directly. Why wrap a checkbox into a div rather than just a ? why did you skin the cat that way when there's 98 other ways to do it? maybe there's a logical reason more than "that's just what came to mind", maybe not. that's the beauty of the whole thing. there is no one way
I'm not sure that argument holds here, not every solution in programming is equally good just because. A element has the advantage of semantically actually representing a progress bar, which improves accessibility and who knows what other browser behavior and heuristics. In general you should try to use semantically correct HTML tags unless you have a really good reason not to. Maybe in this case he couldn't get the appearance he wanted using , which would be an acceptable reason.
Obviously this is nitpicking and this UI was made for fun rather than being a serious library intended for production. But that's a different criticism of the GP than the one you made.