Commit activity
Last 12 weeks · 0 commits
Community health
1 of 6 standards met
Recent PRs & issues
Currently, we're using console.log and do nothing if it's not present. Do we want to inject a `.toString()else` clause all together)? From todo comment: Create alternative logging See code here: https://github.com/addyosmani/myQuery/blob/52185d2f8183d08a05837798c74a7d4cbf485873/myQuery.js#L232
From todo comment: Could probably be a one-time thing and check if a private var With this, anything triggered after window event won't be invoked See code at: https://github.com/addyosmani/myQuery/blob/52185d2f8183d08a05837798c74a7d4cbf485873/myQuery.js#L106
From todo comment: A bit rough, needs more work (probably) See code here: https://github.com/addyosmani/myQuery/blob/52185d2f8183d08a05837798c74a7d4cbf485873/myQuery.js#L119
Recent fixes
I'd like for us to distill down the todos that are left to be completed for this library out into their own issues. We can then easily pick up issues when we get a chance and track progress of how far we are to getting everything wrapped up :) May also be able to get some other contributors involved, but hopefully pushing issues will be a good start. Agree?
A large chunk of code was indented 8 spaces instead of the 4 that was used throughout the rest of the source file.
Encapsulated addListener and removeListener events in a closure to ensure methods aren't invoked multiple times. This is my first ever fork, so PLEASE be kind. lol :)
Repository: addyosmani/myQuery. Description: A JavaScript DOM manipulation library for learning purposes Stars: 10, Forks: 7. Primary language: JavaScript. Languages: JavaScript (100%). Homepage: http://addyosmani.com Open PRs: 0, open issues: 3. Last activity: 14y ago. Community health: 28%. Top contributors: addyosmani, peol, trevnorris, benbabics.