FlaggedRevs form styles missing on Minerva skin
Closed, DuplicatePublic
Actions

Assigned To

None

Authored By

	pmiazga
	Jun 25 2019, 11:34 AM

Description

This affects all FlaggedRevs forms, including special pages for general management and the Review forms on pages themselves.

T226143: Minerva: Add support for SkinAfterContent hook provides SkinAfterContent hook support in Minerva skin. The FlaggedRevs is one of the extensions that use the SkinAfterContent hook to inject "Review this revision" form. Previously, this form wasn't visible on Minerva, so these issues were a lot less visible, but once T226143: Minerva: Add support for SkinAfterContent hook gets merged, Minerva will show the unstyled form on some of the pages themselves:

The form looks broken

form elements touch each other
"Accept revision" button is weirdly placed
the Comment input is not visible - it's there, it's focusable, but it looks like there is no element there same color as footer, no borders)

Details

	Subject	Repo	Branch	Lines +/-
	Convert RevisionReviewForm to OOUI	mediawiki/extensions/FlaggedRevs	master	+271 -248

Customize query in gerrit

Related Objects

Mentioned In: T226143: Minerva: Add support for SkinAfterContent hook
Mentioned Here: T191156: Convert FlaggedRevisions to Codex
T226143: Minerva: Add support for SkinAfterContent hook

Event Timeline

pmiazga created this task.Jun 25 2019, 11:34 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 25 2019, 11:34 AM

pmiazga renamed this task from FlaggedRevs UI broken on Mobile to FlaggedRevs "Review this revision" form styles broken on Minerva skin.Jun 25 2019, 11:35 AM

pmiazga mentioned this in T226143: Minerva: Add support for SkinAfterContent hook.

pmiazga added a project: Web-Team-Backlog.Jun 25 2019, 11:56 AM

ovasileva triaged this task as Medium priority.Jun 25 2019, 3:35 PM

ovasileva edited projects, added Web-Team-Backlog (Design); removed Web-Team-Backlog.

I propose we just swap it to use OOUI for the form. That should fix this, and also make people less likely to notice my, er, other change.

Updated the task description, this actually seems to affect every FlaggedRevs form in Minerva, desktop and mobile alike, including special pages.

Also while I'm still pretty sure my above suggestion would fix them all, frankly migrating the FlaggedRevs forms to any core abstraction would probably fix this, and likely quite a few other things in other skins as well. Er.

As a workaround we could just add some really basic form styles (padding/margins, mostly) to Minerva to covet this sort of thing in general, like I did in Timeless (https://gerrit.wikimedia.org/r/c/mediawiki/skins/Timeless/+/504963/6/resources/forms.less). I think it mostly turned out to be fine even with all the other weird assumptions in mw, but definitely don't do the 'wat' thing, that was indeed an awful idea.

Change 519694 had a related patch set uploaded (by Isarra; owner: Isarra):
[mediawiki/extensions/FlaggedRevs@master] Attempt to convert Special:Stabilization to OOUI

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

gerritbot added a project: Patch-For-Review.Jun 28 2019, 7:55 PM

Change 520116 had a related patch set uploaded (by Isarra; owner: Isarra):
[mediawiki/extensions/FlaggedRevs@master] Convert RevisionReviewForm to OOUI

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

Jdlrobson added a subscriber: Ladsgroup.Jul 1 2019, 8:44 PM

Jdlrobson subscribed.

Jdlrobson moved this task from Backlog to Blocked on the MinervaNeue board.Jan 17 2020, 10:52 PM

Restricted Application added a subscriber: Masumrezarock100. · View Herald TranscriptJan 17 2020, 10:52 PM