Last modified: 2014-09-09 04:53:48 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 T53309, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 51309 - VisualEditor: Support creation of auto-numbered external links and to convert them to labeled ones
VisualEditor: Support creation of auto-numbered external links and to convert...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Editing Tools (Other open bugs)
unspecified
All All
: Low enhancement
: VE-deploy-2014-09-11
Assigned To: Bartosz Dziewoński
:
Depends on: 67377
Blocks: ve-linkediting
  Show dependency treegraph
 
Reported: 2013-07-14 03:27 UTC by Matthew Flaschen
Modified: 2014-09-09 04:53 UTC (History)
8 users (show)

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


Attachments

Description Matthew Flaschen 2013-07-14 03:27:29 UTC
The way VisualEditor handles links with square brackets, but no explicit anchor, can be confusing.  These links are generally discouraged on English Wikipedia, but still occasionally used.

An example is https://en.wikipedia.org/w/index.php?title=User:Superm401/Sandbox&veaction=edit&oldid=564175896 .  The wikitext for this is just [http://example.com], which renders as a linked [1] (or whatever number it is in the page)

However, you should edit the latest version of a page for testing this.  When you edit an oldid page, you won't be able to reproduce the "Save page" disable behavior I mention below.

VisualEditor lets you backspace to remove the characters one by one, which does not really reflect the model of how the link works.  After one or more is removed, "Save page" un-greys.  However, when you go to review the changes, it says, "Could not start the review because your revision matches the latest version of this page."

It does let you remove the link entirely, but to do so you need to explicitly remove each character (not just backspace once to delete the whole thing).

Do a fresh load of the edit page (separate test).  Then, click the chain icon next to the link.  Then, press enter twice to exit the link dialog.  This time, review changes works.  The wikitext becomes:

[http://example.com <nowiki>[1]</nowiki>]

It probably should not change the link if you simply enter the link dialog and press enter without changing anything.  Also, forcing it to text like this is inconsistent with what happens if you delete the right bracket (see above).
Comment 1 Matthew Flaschen 2013-07-14 03:31:00 UTC
Also, if you highlight it, then use "Clear formatting", it does nothing.  I'm not sure what it should do (nowiki the URL?), but if it does nothing it should be greyed out.
Comment 2 James Forrester 2014-03-10 19:32:45 UTC
Current status:

You can …

* … edit the target of an auto-numbered external link to a different external link; and 
* … edit the target of an auto-numbered external link to an internal link, and the anchor will automatically be set as the internal link value.


You cannot …

* … create an auto-numbered external link; 
* … convert an auto-numbered external link to a real external link; or
* … "clear formatting" on an auto-numbered external link.

I think the first two are valuable, but the third isn't, because we're planning to remove the function of removing links and other rich styling from the "clear formatting" function button. Re-titling, setting priority.
Comment 3 Matthew Flaschen 2014-03-10 22:32:09 UTC
(In reply to Matthew Flaschen from comment #0)
> VisualEditor lets you backspace to remove the characters one by one, which
> does not really reflect the model of how the link works.

This is fixed.  It now highlights on first backspace, and erases on second, which I think is handled the same as similar things.
Comment 4 James Forrester 2014-03-11 00:43:18 UTC
(In reply to Matthew Flaschen from comment #3)
> (In reply to Matthew Flaschen from comment #0)
> > VisualEditor lets you backspace to remove the characters one by one, which
> > does not really reflect the model of how the link works.
> 
> This is fixed.  It now highlights on first backspace, and erases on second,
> which I think is handled the same as similar things.

Yeah, sorry, forgot to say "we've done a bunch of work on auto-numbered external links since this bug was filed, so most of this is fixed, however…" at the top. :-)
Comment 5 Gerrit Notification Bot 2014-07-01 16:50:43 UTC
Change 143346 had a related patch set uploaded by Bartosz Dziewoński:
Make it possible to create autonumbered external links from link inspector

https://gerrit.wikimedia.org/r/143346
Comment 6 Bartosz Dziewoński 2014-07-01 16:52:38 UTC
(In reply to James Forrester from comment #2)
> You cannot …
> 
> * … create an auto-numbered external link; 

My patch https://gerrit.wikimedia.org/r/143346 implements this.

> * … convert an auto-numbered external link to a real external link; or

You can do that from ve.ui.MWLinkNodeInspector now, I don't know when it was implemented.

> * … "clear formatting" on an auto-numbered external link.
> 
> I think the first two are valuable, but the third isn't, because we're
> planning to remove the function of removing links and other rich styling
> from the "clear formatting" function button.

I don't think treating it as "formatting" makes sense anyway.
Comment 7 James Forrester 2014-07-01 17:55:21 UTC
(In reply to Bartosz Dziewoński from comment #6)
> (In reply to James Forrester from comment #2)
> > You cannot …

[…]

> > * … convert an auto-numbered external link to a real external link; or
> 
> You can do that from ve.ui.MWLinkNodeInspector now, I don't know when it was
> implemented.

Yes, that was fixed a month or so ago with the creation of MWLinkNodeInspector.
Comment 8 Gerrit Notification Bot 2014-09-06 00:10:56 UTC
Change 143346 merged by jenkins-bot:
Make it possible to create autonumbered external links from link inspector

https://gerrit.wikimedia.org/r/143346
Comment 9 Matthew Flaschen 2014-09-09 04:07:52 UTC
Is it possible to convert an auto-numbered link to one with arbitrary text (that's what the second part of the bug title refers to)?
Comment 10 James Forrester 2014-09-09 04:53:48 UTC
(In reply to Matthew Flaschen from comment #9)
> Is it possible to convert an auto-numbered link to one with arbitrary text
> (that's what the second part of the bug title refers to)?

Yes, that's been possible for a while. In https://meta.wikimedia.org/wiki/Tech/News/2014/27?veaction=edit for example, click on one of the auto-numbered links to edit it, then click "add label" to convert it to a "normal" link.

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


Navigation
Links