On 01/04/2014 05:11 PM, Steven Walling wrote:
This is riskier and bigger change, but it's what
people seemed to want
back in the day, when we first started rolling out mw.ui styles in core
for login/signup etc.
The main risk is just that someone uses the wrong class, or misses one,
so it's (more) inconsistent or looks a little weird. The risk of making
it unusable is much lower.
The requirement for doing this is that we are /much
more diligent/ about
doing a design review of every interface this patch touches and looking
for major flaws that would impair use. If we're going to move forward
with that patch, I want us to not push this out hastily. Let's make a
checklist for design review to work through systematically on Beta Labs
and also perhaps a separate Labs instance.
Probably, a separate one would make more sense, since it has to be in
master to make Beta Labs, which means it will be deployed in a few days.
Matt/Juliusz/anyone: is applying mw.ui to HTMLForm
*required* to
merge 52169 and apply the buttons to EditPage, History, Logs, etc.?
https://gerrit.wikimedia.org/r/#/c/52169 could potentially exclude
HTMLForm (as a way to split the patch), though it should be noted that
HTMLForm already has an opt-in vform option.
I am thinking maybe it might be good to break the
HTMLForm work in to a
separate and simultaneous patch, just because the design audit required
seems much more varied for HTMLForm than edit, history, logs, revdelete,
and so on. Just an idea.
That's reasonable to think about. I posted on the patch.
Matt Flaschen