Created
February 25, 2012 22:57
-
-
Save michaelcullum/1911428 to your computer and use it in GitHub Desktop.
IRC Log for notification boxes
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[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