Last modified: 2011-02-08 21:56:24 UTC
Results of skin tests on SVN head, but expected 1.17 will be all but the same:
Chick - Fine
Classic (Standard) - Sidebar below content
Cologne Blue - Sidebar below content
Modern - Fine
Monobook - Fine
MySkin - Sidebar below content, but is also like that on 1.16wmf4 http://en.wikipedia.org/wiki/Main_Page?useskin=myskin
Nostalgia - Fine
Simple - Fine
Vector - Fine
So for me, 2, possibly 3 skins with the same issues, that need looking at for 1.17
I remember the sidebar-below-content issue, and it had a particularly nasty cause, something to do with user preferences-based CSS. I'll poke at this when I have time, but I won't have much time this week I'm afraid.
Created attachment 7969 [details]
repair quickbar of standard skin
Patch fixes this for standard. The problem is that cologneblue uses different CSS for the same option. Is it safe to introduce skin-dependant behavior into ResourceLoaderUserOptionsModule.php ?
(In reply to comment #2)
> Created attachment 7969 [details]
> repair quickbar of standard skin
> Patch fixes this for standard. The problem is that cologneblue uses different
> CSS for the same option. Is it safe to introduce skin-dependant behavior into
> ResourceLoaderUserOptionsModule.php ?
Safe, yes, good idea, no. Isn't there some existing Skin method that returns this #quickbar stuff?
Mostly fixed in r80034
The CSS is now flipped, because the rest of the CSS is flipped as well. This means that default behavior for rtl implementations has changed from left aligned quick bar to right aligned quickbar.
Might require message update for the preferences ?
Simple mostly done in r80163. Still requires porting of the highlightbroken user option, probably in the same style as done in r80034
(In reply to comment #5)
> Simple mostly done in r80163. Still requires porting of the highlightbroken
> user option, probably in the same style as done in r80034
Was this done yet? What remains to be done for this bug?
Roan, is it possible you could check see if a problem still exists?
I'll try to look at this today.
r80771 fixes the Simple skin.
r80772 fixes Chick skin
r80776 ports Modern to RL
All SkinTemplate generation skins should now be ported.
Stuff that is still broken:
* IE stylesheets of Monobook and Chick
* All non-SkinTemplate skins, although working, are not yet ported to RL
Stuff that needs cleaning up:
* RTL stylesheets: monobook/rtl.css, modern/rtl.css
* myskin/main.css (empty, see no need to keep it)
With r80785, all skins should be ported to ResourceLoader
* common_rtl.css needs to be taken care of.
These rtl files of the skins seem to contain mostly redundant declarations atm, with the exception of some styling for lists under older gecko browsers. Perhaps factor this all into one file and only include when in RTL mode ?
(In reply to comment #11)
> r80776 ports Modern to RL
Is this needed for Modern to work in 1.17, or can it be left out of the release?
> Stuff that is still broken:
> * IE stylesheets of Monobook and Chick
At least Monobook should be fine in 1.17. Not sure about Chick.
Could you (or someone) identify which skins are broken in REL1_17 right now so I know which fixes to backport?
No, these skin ports are not needed for 1.17, they are just ports to RL. I think I marked everything 1.17 that needs porting in order to make the skins work.
(In reply to comment #14)
> No, these skin ports are not needed for 1.17, they are just ports to RL. I
> think I marked everything 1.17 that needs porting in order to make the skins
I say we can mark this as fixed then. If someone spots an error in the 1.17 branch for these skins, they can reopen.