[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 Derk-Jan Hartman changed: What|Removed |Added CC||hartman.w...@gmail.com --- Comment

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 --- Comment #9 from Tim Starling --- One problem with alert() is that it's not HTML, so you couldn't click on the source file links which appear in the Scribunto backtraces. (In reply to comment #1) > Okay this doesn't look so bad now as it se

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 --- Comment #8 from Ryan Kaldari --- OOjs is in core, but if you want to make dialogs, you probably need OOjs UI as well, which is still only in VisualEditor. -- You are receiving this mail because: You are the assignee for the bug. You are o

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 --- Comment #7 from Brad Jorsch --- (In reply to comment #4) > * Remove the dialog code from Scribunto and just use a JS alert in the > meantime. Alerts are pretty unfriendly (since they take over the browser, > stop > JS execution, etc.), but

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 --- Comment #6 from Jon --- In case I was clear please do not follow Ryan's suggestion of "Make jquery.ui and jquery.ui.dialog loadable on mobile by giving them the 'mobile' target in the module definition (and make sure they are only loaded w

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 --- Comment #5 from Jon --- jQuery UI != jQuery Use jQuery by all means of a jQuery plugin for this - you do not need a huge library the size of jQuery UI. jQuery UI is totally unsuitable for mobile. Please do not load it to simply use I'd gue

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 --- Comment #4 from Ryan Kaldari --- For desktop, jQuery UI is fine (at least until OO.js is more widely adopted). The only problem is that mobile doesn't want to load jQuery UI unless absolutely necessary, and Scribunto is used on both desktop

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-22 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 Brad Jorsch changed: What|Removed |Added Priority|Normal |Low --- Comment #3 from Brad Jorsch ---

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 This, that and the other changed: What|Removed |Added CC||at.li...@live.com.au --- Co

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 Andre Klapper changed: What|Removed |Added Keywords||performance Priority|Unprior

[Bug 59808] Scribunto pulls in entire jQuery UI library for error reporting

2014-01-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=59808 Jon changed: What|Removed |Added Summary|Uncaught Error: Unknown |Scribunto pulls in entire |dep