[Bug 25362] Resource loader simple concatenation mode or minification options
https://bugzilla.wikimedia.org/show_bug.cgi?id=25362 Krinkle changed: What|Removed |Added Blocks|24415 | -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 25362] Resource loader simple concatenation mode or minification options
https://bugzilla.wikimedia.org/show_bug.cgi?id=25362 Krinkle changed: What|Removed |Added Status|NEW |RESOLVED CC||krinklem...@gmail.com Resolution||WONTFIX --- Comment #7 from Krinkle 2011-02-01 14:33:52 UTC --- In production mode files are combined into a single request. Whether or not the amount of lines is preserved doesn't help since the files are combined into one. I have yet to see the first major minification system that keeps in mind debugging processes as that's sortof the opposite of attempting to serve files as quickly and effeciently as possible. debug=true mode will serve each individual file seperately (like it used to be before resource loader), I think if you expect a bug reporter to be able to get an actual line number from a particular file, expecting them to append debug=true to the URL should not be a problem. Marking as WONTFIX since this is not the intention of production mode. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 25362] Resource loader simple concatenation mode or minification options
https://bugzilla.wikimedia.org/show_bug.cgi?id=25362 MZMcBride changed: What|Removed |Added Blocks||24415 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 25362] Resource loader simple concatenation mode or minification options
https://bugzilla.wikimedia.org/show_bug.cgi?id=25362 --- Comment #6 from Trevor Parscal 2010-09-29 22:29:27 UTC --- Please see my wikitech-l post about this. http://lists.wikimedia.org/pipermail/wikitech-l/2010-September/049725.html -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 25362] Resource loader simple concatenation mode or minification options
https://bugzilla.wikimedia.org/show_bug.cgi?id=25362 --- Comment #5 from Nux 2010-09-29 22:19:44 UTC --- I'm not saying I'm preserving line numbers. But the code changes: /* something */ to // e...@line[original line number] So I can go to the line number very fast, but I also can view the code and see what is wrong. I can also debug it without switching to debug mode while still having one file instead of many and (in my case) even after simple minification the file is almost 40% shorter (yes I write excessive comments ;-)). -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 25362] Resource loader simple concatenation mode or minification options
https://bugzilla.wikimedia.org/show_bug.cgi?id=25362 --- Comment #4 from Trevor Parscal 2010-09-29 21:55:34 UTC --- Removing documentation only, as you are wanting to do, will not preserve line numbers. The first resource (there can be many resources in a module) in the first module (there can be many modules in a request/response) could have matching line numbers, but every resource after that will not. You must use ?debug=true in the URL of the page or $wgResourceLoaderDebug = true in your LocalSettings.php file to achieve matching line numbers. This will bypass all minification and combination steps, resulting in responses which are easier for developers to work with. Again, this mixed approach does not solve your use case as well as debug=true, thus the middle-ground approach being suggested here is not worth the complexity. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 25362] Resource loader simple concatenation mode or minification options
https://bugzilla.wikimedia.org/show_bug.cgi?id=25362 --- Comment #3 from Nux 2010-09-29 21:40:29 UTC --- Use case is attached ;-). I prefer to have documentation removed to avoid unnecessary load for all users (including those debugging code) and still be able to get a line number of the original code. Yes you will probably say this won't happen for Wikipedia, but I'd like to have such an option e.g. for my office wiki. Currently I had to write my own loader. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 25362] Resource loader simple concatenation mode or minification options
https://bugzilla.wikimedia.org/show_bug.cgi?id=25362 --- Comment #2 from Roan Kattouw 2010-09-29 16:40:09 UTC --- (In reply to comment #1) > Maybe you could help me understand the use case for this. I am personally > opposed to adding unnecessary complexity and configuration to software, so if > there's not a strong use case for it, that constitutes a strong argument > against it. > > I am also against the changes that have been made to JSMin.php recently which > remove all white-space except for new lines (horizontal only, as you are > calling it here). > > My argument is that there should be 2 modes, debug off and debug on. > > * Debug off should be as high performance as possible (stripping every > non-critical character, combining when appropriate, gzip compressed, etc.) > * Debug on should be as easy for developers as possible (keeping line and > character offsets accurate, never combining, etc.) > > Anything between these two extremes will neither server production nor > development well enough to have any value. I totally agree with this entire comment. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 25362] Resource loader simple concatenation mode or minification options
https://bugzilla.wikimedia.org/show_bug.cgi?id=25362 --- Comment #1 from Trevor Parscal 2010-09-29 16:33:48 UTC --- Maybe you could help me understand the use case for this. I am personally opposed to adding unnecessary complexity and configuration to software, so if there's not a strong use case for it, that constitutes a strong argument against it. I am also against the changes that have been made to JSMin.php recently which remove all white-space except for new lines (horizontal only, as you are calling it here). My argument is that there should be 2 modes, debug off and debug on. * Debug off should be as high performance as possible (stripping every non-critical character, combining when appropriate, gzip compressed, etc.) * Debug on should be as easy for developers as possible (keeping line and character offsets accurate, never combining, etc.) Anything between these two extremes will neither server production nor development well enough to have any value. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l