Page MenuHomePhabricator

OOUI Popup should use `drop-shadow` where applicable
Closed, ResolvedPublic

Assigned To
Authored By
Volker_E
Apr 5 2019, 11:06 PM
Referenced Files
F28594094: image.png
Apr 8 2019, 3:56 AM
F28594091: image.png
Apr 8 2019, 3:56 AM
F28594099: image.png
Apr 8 2019, 3:56 AM
F28589448: image.png
Apr 5 2019, 11:06 PM

Description

Popups currently, as of v0.31.3, rely on box-shadow, which is quite well-supported, but implies a UI glitch, as the shadow is rendered only *drumroll* on boxes.
The callouts are not covered by it.

Proposal

Use [[ https://caniuse.com/#feat=css-filters | filter: drop-shadow ]] where applicable in progressive enhancement manner.

v0.31.3Proposed
WikimediaUI theme
image.png (338×690 px, 37 KB)
image.png (192×684 px, 20 KB)
Apex theme
image.png (216×700 px, 20 KB)
image.png (210×700 px, 22 KB)

Event Timeline

Krinkle subscribed.

This is about the Popup element in OOUI, it is not about the MediaWiki extension internally called "Popups".

Krinkle renamed this task from Popups should use `drop-shadow` where applicaple to OOUI Popup should use `drop-shadow` where applicaple.Apr 5 2019, 11:44 PM
Jdforrester-WMF renamed this task from OOUI Popup should use `drop-shadow` where applicaple to OOUI Popup should use `drop-shadow` where applicable.Apr 5 2019, 11:45 PM

Change 501988 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] PopupWidget: Progressively enhance to use filter: drop-shadow()

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

Volker_E triaged this task as Medium priority.Apr 8 2019, 3:56 AM
Volker_E added a project: CSS.
Volker_E updated the task description. (Show Details)
Volker_E moved this task from Backlog to Reviewing on the OOUI board.

Change 501988 merged by jenkins-bot:
[oojs/ui@master] PopupWidget: Progressively enhance to use filter: drop-shadow()

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

Volker_E claimed this task.
Volker_E moved this task from Reviewing to OOUI-0.31.4 on the OOUI board.
Volker_E edited projects, added OOUI (OOUI-0.31.4); removed OOUI.
Volker_E removed a project: Patch-For-Review.

Change 504473 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/core@master] Update OOUI to v0.31.4

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

Change 504473 merged by jenkins-bot:
[mediawiki/core@master] Update OOUI to v0.31.4

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