search

sihorton / appjs-TiddlyWiki

TiddlyWiki plugin for AppJS
5 stars 0 forks source link

Interesting discussion #13

Closed JohnHind closed 12 years ago

JohnHind commented 12 years ago

I have an interesting discussion going with Jeremy Ruston here:

https://github.com/TiddlyWiki/tiddlywiki/pull/109/files

Thought you might like to monitor or join in!

sihorton commented 12 years ago

What do you think, should we implement a refactor of mozillaSave, ieSave, javaSave to a backend object and then submit it as a patch to TiddlyWiki? It is a bigger change than the allowSave change but it does make it simple to implement new save backends.

JohnHind commented 12 years ago

I think we should wait for a steer from Jeremy. No point in spending time on it unless he is likely to accept it. He did indicate he'd accept the allowSave patch, but I can see he might be reluctant to accept a more comprehensive refactor given the stage TW2 is at in its lifecycle and the extensive ecosystem it has in terms of plugins that might get broken if we were not very careful (for example, can we be sure someone's plugin has not used mozillaSave directly?).

He did express interest in building AppJS support into TW5 and may be thinking about a more modular system for that.