Page MenuHomePhabricator

Watchlist Expiry: fix pop-up height issue [MEDIUM]
Closed, ResolvedPublic

Description

As a watchlist expiry user, I want the pop-up behavior to be updated, so that it properly displays all options for long article titles.

Background: This is a follow-up to issues uncovered in T249262, which we would like to address separately.

Acceptance Criteria:

  • Make pop-up adjustable, so that if the article title is long, it shows the full options in the drop-down (rather than cutting off "6 months"

Event Timeline

ifried renamed this task from Watchlist Expiry: fix pop-up width issue to Watchlist Expiry: fix pop-up width issue [MEDIUM].Jul 24 2020, 1:58 AM
ifried renamed this task from Watchlist Expiry: fix pop-up width issue [MEDIUM] to Watchlist Expiry: fix pop-up height issue [MEDIUM].Jul 27 2020, 5:15 PM

Change 616642 had a related patch set uploaded (by HMonroy; owner: HMonroy):
[mediawiki/core@master] Fix pop-up height for watchlist expiry

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

Remember to backport to REL1_35 as well since the commit that caused the regression was also backported (T249262).

Change 616642 merged by jenkins-bot:
[mediawiki/core@master] Fix pop-up height for watchlist expiry

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

Change 616944 had a related patch set uploaded (by MusikAnimal; owner: HMonroy):
[mediawiki/core@REL1_35] Fix pop-up height for watchlist expiry

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

Change 616944 merged by jenkins-bot:
[mediawiki/core@REL1_35] Fix pop-up height for watchlist expiry

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

dom_walden subscribed.

The dropdown now appears fine for a title 255 characters long (the limit).

255_popup.png (280×453 px, 21 KB)

I don't know where else the notification widget is used, so I did not test if the CSS change will impact other UI components.

Test environment: https://en.wikipedia.beta.wmflabs.org MediaWiki 1.36.0-alpha (72ac5e1) 06:44, 31 July 2020

This looks good, based on the screenshot example provided above. I'm marking this as Done.