Page MenuHomePhabricator

[MEX][QC] Add support for displaying multiple violations and bad-parameter constraint check failures
Open, Needs TriagePublic

Description

T411608 implemented support for showing details of QualityConstraints check failures in a popover in the wbui2025 interface. The currently implementation can only show a single issue - if there are multiple issues, it shows only one - and cannot display 'advanced issues' (which are only displayed if multiple issues are present in the existing desktop implementation).

Complete the implementation of multiple-issues pop-ups per the figma designs:

Screenshot 2025-12-02 at 15.14.38.png (1×2 px, 251 KB)

Screenshot 2025-12-02 at 15.14.56.png (1×1 px, 181 KB)

FIGMA LINK

Acceptance Criteria

  • Display of single violations works as previously implemented
  • If multiple violations are present, the multistep dialog replaces the existing pop-up, and allows users to click through the details of each of the violations
    • A single icon is displayed for the multiple violations
    • The icon used is the icon for the most "severe" violation
  • bad-parameter / advanced issues are supported and displayed in the dialog.

Related Objects

Event Timeline

Change #1228431 had a related patch set uploaded (by Audrey Penven; author: Audrey Penven):

[mediawiki/extensions/Wikibase@master] [WIP] Multiple issues in popover

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

Change #1228434 had a related patch set uploaded (by Audrey Penven; author: Audrey Penven):

[mediawiki/extensions/WikibaseQualityConstraints@master] [WIP] show multiple issues in wbui2025

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

Change #1230966 had a related patch set uploaded (by Lucas Werkmeister (WMDE); author: Lucas Werkmeister (WMDE)):

[mediawiki/extensions/Wikibase@master] POC: Make popover contents a list of refs of lists

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

Change #1234333 had a related patch set uploaded (by Audrey Penven; author: Audrey Penven):

[mediawiki/extensions/Wikibase@master] Clean up backwards compatibility for WBQC popover html

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

Change #1228431 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Multiple issues in popover

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

Change #1230966 abandoned by Lucas Werkmeister (WMDE):

[mediawiki/extensions/Wikibase@master] POC: Make popover contents a list of refs of lists

Reason:

superseded by Ibd71713995 (I think sooner or later a reactive version of the constraint indicators would still be useful, but we can figure that out later)

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

Change #1228434 merged by jenkins-bot:

[mediawiki/extensions/WikibaseQualityConstraints@master] Show multiple issues in a popover in wbui2025

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

Change #1234333 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Clean up backwards compatibility for WBQC popover html

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

Change #1238404 had a related patch set uploaded (by Audrey Penven; author: Audrey Penven):

[mediawiki/extensions/WikibaseQualityConstraints@master] Tidy tests related to wbui2025

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

Change #1238404 merged by jenkins-bot:

[mediawiki/extensions/WikibaseQualityConstraints@master] Tidy tests related to wbui2025

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