Page MenuHomePhabricator

Clash between rounded corners and text content is jarring for the category editor in VisualEditor when using the OOjs UI MediaWiki theme
Closed, ResolvedPublic1 Estimated Story Points

Description

See screenshot:

Screen_Shot_2014-12-09_at_11.24.10.png (524×802 px, 52 KB)

(A problem in Apex too, but feels worse in MW.)

Event Timeline

Jdforrester-WMF raised the priority of this task from to Needs Triage.
Jdforrester-WMF updated the task description. (Show Details)
Jdforrester-WMF changed Security from none to None.
Jdforrester-WMF subscribed.

Why aren't we just using a standard mw-ui form for this?

I agree, lets use the standard form field for this.

Why aren't we just using a standard mw-ui form for this?

Nothing in VisualEditor or the new OOjs UI front-end has anything to do with MediaWiki UI, which was built as an ad hoc provision for systems not yet using OOjs UI.

Whatever you call it, (nearly) everything else in VE is using the standard form style first used in MediaWiki UI, while this is using something different and therefore looks weird and silly.

(The category selector really should be implemented using the kind of multiselect widget used here by Phabricator for lists of projects or people subscribed to the task, etc. (Phab calls them "tokenizers", which if you ask me is a pretty silly name.) Some of MediaWiki's forms also use something similar, e.g. on https://wikitech.wikimedia.org/wiki/Special:NovaSecurityGroup (alas, you need to be logged in to view that page, apparently).)

Change 183486 had a related patch set uploaded (by Esanders):
Fix styling of MWCategoryWidget to match MW theme

https://gerrit.wikimedia.org/r/183486

Patch-For-Review

The rounded corners are just left over the from Apex skin where they looked less out of place. Fixed in patch above.

Change 183486 merged by jenkins-bot:
Fix styling of MWCategoryWidget to match MW theme

https://gerrit.wikimedia.org/r/183486

@Ricordisamoa, since you subscribed the same minute that @Jdforrester-WMF resolved this (and removed your subscription in the process), I'm assuming it was just Phabricator's lack of conflict detection. See T78236

I checked the resolution to this bug on deployment-prep and found that to close the category popup you actually have to click outside the button which opened it. But I would expect that clicking the button again (while the popup is still open) would close it.

I checked the resolution to this bug on deployment-prep and found that to close the category popup you actually have to click outside the button which opened it. But I would expect that clicking the button again (while the popup is still open) would close it.

Open a ticket?

Checked in betalabs - see the attached screenshot.

no_clash_with_round_corners.png (416×730 px, 48 KB)

Verified the fix in Betalabs and test2