Skip to content

Instantly share code, notes, and snippets.

@michaelcullum
Created February 25, 2012 22:57
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save michaelcullum/1911428 to your computer and use it in GitHub Desktop.
Save michaelcullum/1911428 to your computer and use it in GitHub Desktop.
IRC Log for notification boxes
[2012-02-15 13:39:34] <igorw> poke imkingdavid, brunoais, callumacrae, nn-, unknownbliss
[2012-02-15 13:39:49] <imkingdavid> ?
[2012-02-15 13:39:55] <igorw> last thing I need to on ajax is error handling
[2012-02-15 13:39:56] <brunoais> what do ya need :)?
[2012-02-15 13:39:57] <unknownbliss> yep
[2012-02-15 13:40:41] <unknownbliss> I need a website & email for you and nils for meta-data file for his sample ext
[2012-02-15 13:41:07] <igorw> naderman@naderman.de and igor@wiedler.ch I think
[2012-02-15 13:41:11] <igorw> website is optional
[2012-02-15 13:41:17] <igorw> you can put naderman.de for nils
[2012-02-15 13:41:21] <unknownbliss> ok
[2012-02-15 13:41:44] <imkingdavid> i'm not familiar with ajax/javascript error handling...
[2012-02-15 13:41:45] <unknownbliss> all optional but best to have as much as possible so its a better example
[2012-02-15 13:41:52] <unknownbliss> ^^ what he said
[2012-02-15 13:41:59] <brunoais> I am
[2012-02-15 13:42:31] <igorw> ok, so the first question is: do we allow users to handle ajax errors?
[2012-02-15 13:42:52] <igorw> imo we should allow it if they call ajaxify manually only
[2012-02-15 13:42:57] <brunoais> I think it should depend on the error
[2012-02-15 13:43:02] <unknownbliss> thanks
[2012-02-15 13:43:10] <brunoais> ex:
[2012-02-15 13:43:28] <igorw> and the next question obviously is: what do we do in the UI on error?
[2012-02-15 13:43:35] <brunoais> if connection failed, check if browser is offline. If not, try again, try again, tell the user the error
[2012-02-15 13:43:58] <igorw> hm, imo not worth it
[2012-02-15 13:44:03] <igorw> that's just polishing
[2012-02-15 13:44:28] <igorw> I just want to give some basic feedback that doesn't look like crap
[2012-02-15 13:44:39] <brunoais> warning the user should be a div that is fixed in some place where it won't bother the user much and would call the user's attention at the same time
[2012-02-15 13:44:52] <igorw> yea, shouldn't be interruptive
[2012-02-15 13:45:01] <brunoais> something with jQuery's fade in, ex
[2012-02-15 13:45:02] <igorw> the problem is when it's at the top of the page and he scrolled, he won't see it
[2012-02-15 13:45:11] <brunoais> I said
[2012-02-15 13:45:15] <brunoais> it should be fixed
[2012-02-15 13:45:21] <brunoais> not absolut fixed
[2012-02-15 13:45:31] <brunoais> *not absolut placed
[2012-02-15 13:45:37] <igorw> maybe something like this
[2012-02-15 13:45:37] <igorw> http://wavded.github.com/humane-js/
[2012-02-15 13:45:38] <brunoais> its placement should be fixed
[2012-02-15 13:46:55] <igorw> anyone have a better idea?
[2012-02-15 13:47:22] <igorw> or just use our alert thing
[2012-02-15 13:47:27] <brunoais> that is not a bad idea. Need to check its options, though
[2012-02-15 13:47:29] <igorw> I mean it's not like it will happen every day
[2012-02-15 13:47:30] <brunoais> anyway
[2012-02-15 13:47:38] <igorw> it's ugly and interruptive
[2012-02-15 13:47:46] <igorw> but at least it's some feedback
[2012-02-15 13:48:02] <igorw> and I could add it in 1 line of code
[2012-02-15 13:48:17] <igorw> or maybe 5
[2012-02-15 13:48:19] <brunoais> the theme I recommend is jackedup(I prefer) or libnotify
[2012-02-15 13:48:30] <igorw> yes, jackedup is the only decent one
[2012-02-15 13:48:34] <igorw> used it on packagist
[2012-02-15 13:48:46] <igorw> just in case you'd get an error, which you never do btw :P
[2012-02-15 13:51:29] <igorw> brunoais: any ideas?
[2012-02-15 13:51:57] <brunoais> hum...
[2012-02-15 13:52:16] <brunoais> jgrowl maybe
[2012-02-15 13:52:24] <brunoais> give it a peek
[2012-02-15 13:52:44] <igorw> the thing is if I want to add some js lib we need to discuss it, etc etc
[2012-02-15 13:53:01] <igorw> ugh, growl is annoying :P
[2012-02-15 13:53:22] <brunoais> k
[2012-02-15 14:16:34] <igorw> phpbb.alert it is then
[2012-02-15 14:17:57] <brunoais> I don't even know how phpbb.alert looks like :S
[2012-02-15 14:18:06] <brunoais> any img available?
[2012-02-15 14:27:56] <marc1706> igorw: I actually like jQuery UI notification widget: http://www.erichynds.com/examples/jquery-notify/
[2012-02-15 14:28:49] <igorw> unknownbliss: the thing with package.json is that node uses that as well
[2012-02-15 14:29:06] <igorw> which might be a bit confusing to people who know this
[2012-02-15 14:29:13] <unknownbliss> ah
[2012-02-15 14:29:42] <unknownbliss> how about extension.json?
[2012-02-15 14:29:53] <igorw> then we cannot use it for language and styles
[2012-02-15 14:30:06] <igorw> without adding the check everywhere
[2012-02-15 14:30:26] <unknownbliss> http://www.erichynds.com/examples/jquery-notify/ - would be nice for flash messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment