Page MenuHomePhabricator

[MEX][QC] Open popover when issue icon is clicked
Closed, ResolvedPublic

Description

T411604 introduces icons to the display of statements. These icons should be clickable, and when clicked should open up a popover that explains the significance of the icon.

The contents of the popover should be defined by the WikibaseQualityConstraints extension - there will need to be a mechanism to associate popover design / HTML content with the icon when icons are added to the pinia store for a snak.

Screenshot 2025-12-02 at 15.14.06.png (1×1 px, 206 KB)

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

  • Popovers can be associated with icons and when the icons appear next to snaks, clicking on the icons shows the popover (which can then be dismissed again)

Notes

  • Depending on the progress with T411606 T411604 and T411602, it may already be possible to test this end-to-end. If not, it suffices for the purposes of this ticket to introduce the popover mechanism and cover it with unit tests. The final implementation will come in T411609
  • All of the content should be generated (and formatted) by QualityConstraints

Details

Related Changes in Gerrit:

Event Timeline

ArthurTaylor renamed this task from [MEX] Open popover when issue icon is clicked to [MEX][QC] Open popover when issue icon is clicked.Dec 4 2025, 7:21 AM

@Alice.moutinho In the first screenshot, the "Help | Discuss" is not pinned for a single issue, but is pinned in the third screenshot (Overflow). Is that inconsistency expected or is that a mistake?

Hi @ArthurTaylor, it is not a mistake, and i explain why this is so in the original ticket https://phabricator.wikimedia.org/T400676:

The regular popover does not have a footer option, so the links need to be included in the content. The multistep-dialog (for multiple issues) pattern does have a footer option, so we can just put it in there for every issue! hope it helps

Change #1217702 had a related patch set uploaded (by Arthur taylor; author: Arthur taylor):

[mediawiki/extensions/Wikibase@master] Add popover for indicators

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

Change #1217702 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Add popover for indicators

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