Page MenuHomePhabricator

Consistently style links between read mode VisualEditor modes (internal, inter-wiki and external)
Open, Needs TriagePublic

Description

I've recently seen a considerable amount of inconsistency in the ways in which the different types of links (internal, inter-wiki and external) are styled in various cases such as,

  1. Rendered wikitext
  2. VisualEditor
  3. 2017 wikitext editor preview

I searched for the related tasks and found a few of them but there wasn't anything that collects the different issues together and not every issue was covered. (I'll link the tasks I found to be most related, below.) So I thought of creating a task that collects them together. So, the tagging might be incorrect correct them in case they aren't right.

Wikitext

links confusion (wikitext).png (233×1 px, 12 KB)

= Links =

=== Internal link ===
[[Special:Random]]

=== Inter-wiki link ===
[[m:Parsoid/Tests:quotes]]

[[dict:foo]]

=== External link ===
[https://example.com External link]

Different stylings

Read mode renderingVisualEditor rendering2017 wikitext editor previewStandalone visual diff
Screenshot
link confusion (rendering).png (288×1 px, 14 KB)
links confusion (VE).png (305×1 px, 13 KB)
links confusion.png (293×1 px, 13 KB)
image.png (333×1 px, 14 KB)
Internal linksNo icon No icon No icon No icon
Inter-wiki linksNo icon With icon No icon No icon
External linksWith icon With icon No icon No icon
NotesAll other modes must match read mode rendering.Partially fixed by https://gerrit.wikimedia.org/r/508693T187571; fixed by T58756T187692; not fixed?

Environment details

BrowserFirefox Nightly 59.0a1
OSDebian GNU/Linux 9 (stretch)
SkinVector (default)

Related tasks

Event Timeline

Schnark subscribed.

The missing icon after external links in NWE preview is definitely a regression, in T180214 it was still there.

This is related to T58756. Depending on how that is resolved this is probably duplicate.

The NWE preview issues should be fixed now.

However, VE showing external links for interwiki links isn't fixed because VE doesn't know about interwiki links .. it needs to convert the link to Wikitext (by asking Parsoid) and only then will it know if the "external link to another wiki" is actually an interwiki link.

I am going to consider the Parsoid side of this issue resolved and will untag Parsoid.

JTannerWMF subscribed.

Can someone determine whether or not this is resolved?

The Parsoid side is finished, but we still render interwiki links as external in VE.

Change 508693 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/VisualEditor@master] Fix initial rendering of interwiki links

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

"This does not fix the rendering of new or modified interwiki links, which will require an API request."

matmarex subscribed.

Esanders merged a task: T187571: External links look like interwiki links in preview.

I don't think that was actually a duplicate of this task. But it seems to be resolved now, thanks to Parsoid changes from T58756.

Esanders merged a task: T187692: Correctly style external links in visual diffs.

I don't think this is a duplicate either. The proposed patch above does not resolve this issue.

Change 508693 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Fix initial rendering of interwiki links

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

matmarex moved this task from Code review to In progress on the VisualEditor (Current work) board.

I made a pretty table and added a case that covers T187692, since it was merged here.

Krinkle renamed this task from Consistently style links (internal, inter-wiki and external) to Consistently style links between read mode VisualEditor modes (internal, inter-wiki and external).May 22 2019, 9:17 PM

Partially fixed by latest patch. Moving back off current work board as other cases are lower priority.