Sorry for the delay, I've been rather busy as of late!
I think it all looks good! I will tweak a few tiny things after merging- you can check my commits in a bit to see what I did.
I missed your edit to your last request asking about jQuery- I generally prefer to only use libraries in cases where we can utilize almost all of their features, or if it saves us time on very oft-performed tasks. I mostly want to avoid detriments to page load time, and we do have a lot of mobile users. jQuery's 84.63 KB isn't too bad, but, given that we're only using features that have well-enough supported alternatives (querySelector, classList), I'm in favor of replacing jQuery with these native counterparts.
Sorry for the delay, I've been rather busy as of late!
I think it all looks good! I will tweak a few tiny things after merging- you can check my commits in a bit to see what I did.
I missed your edit to your last request asking about jQuery- I generally prefer to only use libraries in cases where we can utilize almost all of their features, or if it saves us time on very oft-performed tasks. I mostly want to avoid detriments to page load time, and we do have a lot of mobile users. jQuery's 84.63 KB isn't too bad, but, given that we're only using features that have well-enough supported alternatives (querySelector, classList), I'm in favor of replacing jQuery with these native counterparts.