User "Dantman" posted a comment on MediaWiki.r81507.

Full URL: 
https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/81507#c20649
Commit summary:

Completely remove support for legacy style skins. All legacy skinning options 
are now part of a SkinLegacy/LegacySkinTemplate pair that inherits from the 
normal SkinTemplate setup. Also ported our three built in skins to use the new 
legacy classes. ( ;) if you want to kill legacy skins now, you only have to svn 
rm 4 files)

Comment:

I tried to argue for a /trunk/skins/ but never got one, 
/trunk/extensions/skins/ seams to be the only place if you're going to check 
something into svn (I never got around to putting Monaco into svn because I was 
never satisfied that I was finished with monaco-port and svn would make working 
on it too much of a hassle).
Take a look at /trunk/extensions/skins/p2wiki/. I've come up with a pattern for 
packaging skins that keeps them with other skins and lets you drop a folder 
into /skins/ and install instead of having to add both a folder and a php file.

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to