Pasted links with Arabic target display in percent encoding in VE link inspector until saving
Closed, ResolvedPublic1 Estimated Story Points
Actions

Assigned To

Authored By

	nshahquinn-wmf
	Jun 17 2016, 9:03 PM

Description

Copy a wikilink with a target in Arabic script from one VE surface to another.
Click on the pasted link so that the link inspector opens.
The link target displays in percent enconding.

The effect disappears after you save the page. Verified in Firefox 48.0b1 and Chrome 52.0.2743.41 beta (64-bit) on OS X 10.11.5.

Screen Shot 2016-06-17 at 14.02.18.png (390×826 px, 37 KB)

Event Timeline

nshahquinn-wmf created this task.Jun 17 2016, 9:03 PM

Restricted Application added subscribers: Zppix, Aklapper. · View Herald TranscriptJun 17 2016, 9:03 PM

• Elitre subscribed.Jun 18 2016, 5:04 AM

Jdforrester-WMF renamed this task from Pasted links with Arabic target display in percent encoding in VE link inspector to Pasted links with Arabic target display in percent encoding in VE link inspector until saving.Jun 18 2016, 7:30 AM

Jdforrester-WMF triaged this task as Medium priority.

Jdforrester-WMF added a project: VisualEditor-MediaWiki-Links.

Jdforrester-WMF set the point value for this task to 1.

Is this the same as https://gerrit.wikimedia.org/r/#/c/293332/ ?

Esanders added subscribers: DLynch, dchan.Jun 20 2016, 11:43 AM

In T138113#2393366, @Esanders wrote:

Is this the same as https://gerrit.wikimedia.org/r/#/c/293332/ ?

Nope, still in effect in master.

I actually can't reproduce this one.

I grabbed some arabic links from https://ar.wikipedia.org/ (editing the عضو_برلمان article, to be precise), and pasted them into my local mediawiki VE. They correctly pasted as "internal" wiki-links, and showed as non-encoded text.

What am I missing?

Hmm, yeah, I think this got fixed on Beta Cluster since I tested it on Friday night; perhaps a auto-deploy slowness.

Jdforrester-WMF moved this task from To Triage to TR0: Interrupt on the VisualEditor board.Jun 20 2016, 4:46 PM

Yep, looks fixed for me on the beta cluster as well.

	F4175842: Screen Shot 2016-06-17 at 14.02.18.png
	Jun 17 2016, 9:03 PM

	F4185904: pasted_file
	Jun 20 2016, 3:48 PM

Pasted links with Arabic target display in percent encoding in VE link inspector until savingClosed, ResolvedPublic1 Estimated Story PointsActions

Description

Event Timeline

Pasted links with Arabic target display in percent encoding in VE link inspector until saving
Closed, ResolvedPublic1 Estimated Story Points
Actions