notifyjs icon indicating copy to clipboard operation
notifyjs copied to clipboard

Add a "Contributors" section to the documentation

Open jpillora opened this issue 10 years ago • 7 comments

:warning: As many have noticed, this project could do with some work. I'm looking for people to take over the project. If you're interested please leave a comment below. Once we have one or two people, I'll make a Github organisation and move this project there.

Required tasks

  • Resolve outstanding issues
  • CoffeeScript is becoming obsolete - we need to change the source to ES6 via Babel

Future tasks

  • Implement more styles
    • Style changer in the docs
  • Possibly remove jQuery dependency?
    • See http://substack.net/weaning_yourself_off_jquery
    • Will need a small animation library? CSS3 animations?
    • Maybe this will need two builds notify.js (modern) and notify.legacy.js (which includes jqlite)

jpillora avatar Aug 28 '15 08:08 jpillora

Make me a maintainer. I'll take it over.

EvanCarroll avatar Sep 19 '15 00:09 EvanCarroll

Just nuke the coffee script and work with the outputted source. CoffeeScript sucks, and it's not necessary. All of notifyjs-combined is 600 lines. There is no need for this project to have a build system or use coffeescript.

EvanCarroll avatar Sep 19 '15 00:09 EvanCarroll

Hey @EvanCarroll Yep agreed - it should be pure JS. A few notes:

  • I'm using gh-pages instead of master to host the docs/
  • I'm happy for it to just use ES5. Without a build system, how would we achieve https://github.com/jpillora/notifyjs/tree/gh-pages/dist ?

jpillora avatar Sep 19 '15 02:09 jpillora

@EvanCarroll could you please fork and do the switch to JS? I'll review then give you commit access

jpillora avatar Sep 19 '15 02:09 jpillora

@jpillora https://github.com/jpillora/notifyjs/pull/45

EvanCarroll avatar Oct 20 '15 16:10 EvanCarroll

Let's put this in the README.md and start to create real docs for the project and close this. It's not an issue that will go away and every project wants productive people.

EvanCarroll avatar Nov 03 '15 19:11 EvanCarroll

Agreed

jpillora avatar Nov 03 '15 21:11 jpillora