Page MenuHomePhabricator

[MEX] M3.3.2 - editing snaks on references
Closed, ResolvedPublic

Description

As a contributor, i want to be able to edit references and snaks, so i can provide verifiable sources for statements.

Displaying references and snaks on the edit screen:

Screenshot 2025-10-10 at 11.26.42.png (1×1 px, 123 KB)

Figma for display of references and snaks

Adding snaks

Screenshot 2025-10-10 at 11.28.06.png (1×1 px, 189 KB)

Figma file for adding snaks

Deleting references or snaks

Screenshot 2025-10-10 at 11.28.38.png (852×1 px, 157 KB)

Figma file for deletion

Error handling

When a field is incomplete/incorrect, it should display there is an error (desktop behavior)
NEW: Should it be a simple input string, and nothing is written inside, once it is out of focus/ the user moves on from the field, it should also become red/error state. This is not currently implemented on desktop.

Screenshot 2025-09-29 at 17.04.32.png (1×1 px, 156 KB)

Event Timeline

Alice.moutinho updated the task description. (Show Details)

@AudreyPenven_WMDE hey i just swaped the images and corrected the figma files: i had forgotten the snak type editing option in the fields. Is this something we can still mend in this ticket or was i too late (and we should make another ticket to add the snak type selection later)?

@Alice.moutinho all good! I haven't gotten very far with this yet, so it's fine to include it in this ticket.

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

[mediawiki/extensions/Wikibase@master] [WIP] edit snaks on references

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

Lucas_Werkmeister_WMDE renamed this task from [MEX] 3.3.2 - editing snaks on references to [MEX] M3.3.2 - editing snaks on references.Oct 21 2025, 9:03 AM

🔄 Peer Review Handover Summary

Moved from "In Peer Review" > "In Development" due minor code review comments.

✅ Work Completed

  • The acceptance criteria are almost met regarding the proposed development with local manual testing .. looks promising so far (still missing only the error handling logic).

🧪 Testing/Technical Status

Completed:

  • E2E tests written and passing
  • Manual testing performed
  • CI Builds are green and no merge conflicts (so far)
  • Code review received

🚧 Remaining Work

Medium Priority:

  • The mentioned review comments must be addressed and resolved.
  • Only one note on the errors handling (missing implementation .. included in the comment reviews).

🔗 Related Resources

  • Related Gerrit Patch(s):

https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikibase/+/1194628


Handover Date: 30th October 2025
Last Updated By: Mahmoud

@Alice.moutinho @Arian_Bozorg I'm not sure about the Error Handling part .. is it an AC part of this ticket implementation? Or it should be general validation + error handling ticket?

Change #1194628 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Edit and delete snaks on references

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

I created a general error handling ticket here: T408928

I’m not sure why this task is in Ready to Peer Review again, since all attached changes are merged… is Allow adding new snaks to references supposed to be attached to this task?

oops, yeah, that should be attached. I'll fix the commit message

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

[mediawiki/extensions/Wikibase@master] Allow adding new snaks to references

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

Change #1200100 merged by jenkins-bot:

[mediawiki/extensions/Wikibase@master] Allow adding new snaks to references

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