Page MenuHomePhabricator

The dialog to save filters on Recent Changes needs adjustments to better communicate the initial status
Closed, ResolvedPublic

Description

Currently the dialog for saving a filter looks like this:

Screen Shot 2017-06-28 at 11.18.19.png (369×302 px, 14 KB)

There are several aspects that can be improved:

  • The primary action should only be active when the name input field is not empty. Otherwise we provide an active action that cannot be used initially.
  • Adding a placeholder to the input field such as "Describe the purpose of the filters" can help to encourage good naming of the filters the user creates.
  • Adjusting the language to avoid "settings" (T167876) and the icon to use the filled bookmark one (T168828) are part of other tickets, but some of the changes may still be pending for the dialog so make suer to check.

A mockup is shown below and additional context i available in the original ticket (T164128):

RC-QL-dialog-states.png (1,024×309 px, 24 KB)

Event Timeline

Change 361909 had a related patch set uploaded (by Mooeypoo; owner: Mooeypoo):
[mediawiki/core@master] RCFilters: Correct display of save filter popup

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

Change 361909 merged by jenkins-bot:
[mediawiki/core@master] RCFilters: Correct display of save filter popup

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

Change 362268 had a related patch set uploaded (by Mooeypoo; owner: Mooeypoo):
[mediawiki/core@master] RCFilters: Followup I5e3600: Set actual value rather than object

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

Change 362268 merged by jenkins-bot:
[mediawiki/core@master] RCFilters: Followup I5cede8: Set actual value rather than object

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

Change 362308 had a related patch set uploaded (by Mooeypoo; owner: Mooeypoo):
[mediawiki/core@wmf/1.30.0-wmf.7] RCFilters: Followup I5cede8: Set actual value rather than object

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

Checked in betalabs

Adjusting the language to avoid "settings"

Everything else looks according to the mockup.

Screen Shot 2017-06-29 at 3.10.48 PM.png (349×232 px, 16 KB)

Screen Shot 2017-06-29 at 3.16.31 PM.png (354×240 px, 14 KB)

QA Recommendation: Resolve

Change 362308 merged by jenkins-bot:
[mediawiki/core@wmf/1.30.0-wmf.7] RCFilters: Followup I5cede8: Set actual value rather than object

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

Stashbot subscribed.

Mentioned in SAL (#wikimedia-operations) [2017-06-29T23:31:17Z] <catrope@tin> Synchronized php-1.30.0-wmf.7/resources/src/mediawiki.rcfilters/: RCFilters fixes (T169169, T169107, T169042) (duration: 00m 42s)