Last modified: 2012-11-26 23:32:21 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T44282, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 42282 - VisualEditor: Link inspector should edit existing annotation rather than creating new one
VisualEditor: Link inspector should edit existing annotation rather than crea...
Status: RESOLVED WONTFIX
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: Highest normal
: VE-deploy-2012-11-26
Assigned To: Trevor Parscal
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-11-20 02:59 UTC by Roan Kattouw
Modified: 2012-11-26 23:32 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Roan Kattouw 2012-11-20 02:59:06 UTC
When editing an existing link, the link inspector should grab the existing annotation and edit it, rather than starting over with a brand new annotation.

Destroying the existing annotation loses metadata we need for preserving syntactical variation (hrefPrefix and origTitle), which means that currently, simply inspecting a link without actually changing its target can cause that link to be changed in the wikitext.
Comment 1 Trevor Parscal 2012-11-26 19:19:17 UTC
What happens when the annotation type changes from internal to external? We have to create a new object at some point.

What about when someone removes an annotation, then adds it back in? Do we then have a diff?

I'm suspicious about this being the only way we cleanly round-trip.
Comment 2 Trevor Parscal 2012-11-26 23:18:18 UTC
Roan and I agree that as long as the annotation isn't re-applied (and meta data stripped) unless we actually changed the target, this behavior is OK.

The concept is, if you touch something with the editor we have the right to normalize that bit.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links