Page MenuHomePhabricator

Step 2: confirm existing reference when edit doesn't influence it
Open, Needs TriagePublic3 Estimate Story Points

Description

As an editor I want to be able to confirm an existing reference if my edit doesn't invalidate it in order to remove the warning.

Problem:
When fixing a typo in a value for example that does not invalidate the reference then I can only make the warning go away by reloading the page (This will be the situation after completion of T234790).

Screenshots/mockups:

Find specs and more copy explorations in this Figma page.

BDD
GIVEN an edit that changes the value but not the reference
AND the warning icon is visible
WHEN clicking on the warning icon the explanation is shown
AND I click the "Remove warning" button to confirm that reference still matches the value
THEN the popper closes
AND the warning icon goes away

Acceptance criteria:

  • When confirming the reference is fine the warning icon goes away

Details

Related Gerrit Patches:
mediawiki/extensions/Wikibase : masterTR: Add "remove warning" button to popper

Event Timeline

Restricted Application added a project: Wikidata. · View Herald TranscriptOct 7 2019, 8:18 AM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

@Jan_Dittrich @Erdinc_Ciftci_WMDE Can you please create a mockup for this and add it?

As far as I am concerned and according the the flowcharts I send around, this is milestone 2. Are we already there?

As discussed yes. We'll start with discussing milestone 2 stories in story writing on Friday.

Jan_Dittrich updated the task description. (Show Details)Oct 9 2019, 2:38 PM

The above mockup says "remove message" which is pretty much what it does. It does not say "the edit is fine" or an equivalent, since this would probably need more words than I can fit on a button. Also it currently has no fixed semantic meaning, so removing it for one user is what is happening. When we have the 3rd milestone we need a more elaborate UI which gives some explanation for the now shared assertion of fine/not fine, probably in a popup.

I also show a static warning message. However, I find not having it dynamically problematic usability-wise since we do not use the word "value" in our UI, so it is likely that people do not understand the not-dynamic message using the term "value" (Except if they already know "Wikidata-speak", but this is something they need to learn from others, so it is not cool for new users).

Lydia_Pintscher renamed this task from confirm existing reference when edit doesn't influence it to Step 2: confirm existing reference when edit doesn't influence it.Oct 11 2019, 10:31 AM

Change "remove" to "dismiss"

Jan_Dittrich updated the task description. (Show Details)Oct 14 2019, 2:16 PM
WMDE-leszek set the point value for this task to 3.Oct 14 2019, 2:18 PM

Ping @Sarai-WMDE for working more on mockup.

Tarrow moved this task from Ready to pick up to Needs Work on the Wikidata Tainted References board.
Tarrow added a subscriber: Tarrow.

Can you add the mockup details here

Jan_Dittrich added a mock: Restricted Mockup.Dec 19 2019, 11:01 AM

{M290/985}

Sarai-WMDE updated the task description. (Show Details)Dec 23 2019, 10:21 AM
Tarrow updated the task description. (Show Details)Mon, Jan 6, 3:12 PM

Current state: Sarai is working on new mockup.

WMDE-leszek updated the task description. (Show Details)Mon, Jan 13, 3:25 PM
Sarai-WMDE updated the task description. (Show Details)Mon, Jan 13, 3:57 PM
Sarai-WMDE removed Sarai-WMDE as the assignee of this task.Thu, Jan 16, 9:11 AM
Sarai-WMDE updated the task description. (Show Details)
hoo claimed this task.Mon, Jan 20, 12:55 PM
hoo moved this task from To Do to Doing on the Wikidata-Tainted-References-Sprint9 board.

Change 566319 had a related patch set uploaded (by Hoo man; owner: Hoo man):
[mediawiki/extensions/Wikibase@master] TR: Add "remove warning" button to popper

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