Yeah the input case was a kind of weird one as there was undocumented use of mw-ui-input throughout mediawiki in Special:Search and Special:UserLogin (it was in the less file and in usage but not in the style guide). May has suggested linking to live examples in future which I think will help with this.
I think as a rule we should all get into the habit of making MediaWiki ui the de facto place for new components and ensuring documentation of when and how to use is kept to standards. I like the idea of each change requiring a +1 from the main teams that use it (which I think are Flow, Growth and Mobile correct me if wrong). e.g. +3 :) I'm happy to see mobile, Flow and growth caring about the same problems and things getting merged more rapidly then we have done previously. I'm keen to keep this momentum.
Maybe its worth getting together people not going to Wikimania and getting mw-ui-input styling resolved in a way that makes everyone happy and keeps the solution generalised and consistent ?
We may also want to consider a beta concept of mediawiki ui that allows us to hide experimental styling behind additional modifier classes. I'm keen for teams to rapidly prototype and experiment with new design ideas but to do so in a way that keeps compatibility. e.g. mw-ui-beta .mw-ui-button
On 29 Jul 2014 22:08, "Shahyar Ghobadpour" <sghobadpour@wikimedia.org> wrote:
On Tue, Jul 29, 2014 at 4:29 PM, Steven Walling <swalling@wikimedia.org> wrote:I'm glad we merged this sooner rather than later, since it means there are fewer Flow-specific overrides on top of mediawiki.ui and it forces us to have a cross-team discussion.
Flow doesn't use overrides on top of mediawiki.ui (except for mw-ui-button, which only started recently). We wrote all mw-ui components independently of Core, using a flow-ui prefix -- including a rewrite of mw-ui-button, which I felt had become too messy and confusing. When I felt that the flow-ui versions had reached a certain level of maturity, I'd put them for review into Core's gerrit as mw-ui components.On Tue, Jul 29, 2014 at 11:16 PM, Erik Moeller <erik@wikimedia.org> wrote:I agree we have to be wary about consistency. I'm not sure putting
these kinds of changes on the release train incrementally is the way
to go -- perhaps having a test instance in Labs run a WIP changeset or
branch til we're happy with it, including things like RTL testing?This is specifically the reason I was using Flow as the testbed to develop and iterate upon the mw-ui components. However, I think my team jumped the gun on putting this into Core without fully testing all of the use cases. I take responsibility for this, as I should have been more careful to a) include i18n people in the design mocks, and b) thoroughly test any current uses of affected mw-ui components being moved to Core.Going forward, I think we will need to shift from a single +2 in gerrit, to a +1 from several teams (perhaps Flow, Growth, Mobile, and Language), before approving a merge to Core. For these mediawiki.ui design goals to succeed, we'll need to make sure we have our primary bases covered via these teams.--Shahyar_______________________________________________
Design mailing list
Design@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/design
_______________________________________________
Design mailing list
Design@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/design