Hi all,
I'll keep this short. Please read and comment on https://www.mediawiki.org/wiki/Requests_for_comment/LESS and the associated patch: https://gerrit.wikimedia.org/r/#/c/78669/
Muchas gracias,
As long as the change does not inhibit extensions from hooking in and using other CSS pre-processors, I don't see any issue with using LESS in core.
The one other thing I'm concerned about is why we're using Memcached so much for file caching (we use it to cache the results of CSS minification, and now we'd be using it for LESS compulation as well). Once this patch is merged, we'd be caching two-three copies of every static file in MediaWiki in a completely in-memory network cache. Why are we not just putting them in a file cache?
*-- * *Tyler Romeo* Stevens Institute of Technology, Class of 2016 Major in Computer Science www.whizkidztech.com | tylerromeo@gmail.com
On Mon, Aug 19, 2013 at 8:14 PM, Steven Walling swalling@wikimedia.orgwrote:
Hi all,
I'll keep this short. Please read and comment on https://www.mediawiki.org/wiki/Requests_for_comment/LESS and the associated patch: https://gerrit.wikimedia.org/r/#/c/78669/
Muchas gracias,
-- Steven Walling https://wikimediafoundation.org/ _______________________________________________ Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l
On Aug 20, 2013, at 2:31 AM, Tyler Romeo tylerromeo@gmail.com wrote:
As long as the change does not inhibit extensions from hooking in and using other CSS pre-processors, I don't see any issue with using LESS in core.
However if and when we adopt LESS support in core, which only happens if we also incorporate it into our MediaWiki coding conventions, complying extensions will, by convention, not be tolerated to use other pre-processors.
However I agree that ResourceLoader in general should be agnostic and allow implementation and usage of whatever you want in your own extensions.
From quickly looking at the draft patch set and the existing extension[1][2] that already implements this I we can conclude that this is already the case, and I'll hereby vouch for continued support of such extensibility for other pre-processors as well. However core should only (at most) include 1.
-- Krinkle
[1] https://www.mediawiki.org/wiki/Extension:Less [2] https://github.com/wikimedia/mediawiki-extensions-Less
wikitech-l@lists.wikimedia.org