Links, images, tables, and citations added at top instead of position in WikiEditor when syntax highlighting is enabled (on Chrome only)
Closed, ResolvedPublicBUG REPORT
Actions

Description

Hello, there is a bug with link and image buttons of wiki editor in it.wikivoyage. Every time I want to add a link highlighting a word using link button (on top editor), the entire link appear not in the position I want but on top of thread. The same if you add an image using the button of editor. I see this happen only using "syntax highlighting" in source mode.

Steps

Use Google Chrome (other browsers are possible?)
Go to https://en.wikipedia.org/w/index.php?title=Sebastian_Mayer&action=edit with WikiEditor
Click Syntax highlighting in editor toolbar to enable syntax highlighting
Select any word
Click add link item in editor toolbar
Click Add collegemento button in dialog
Line becomes on top, as expected.

Current

The link appears at the top, not instead of the selected fragment.

Expected

The link appears instead of the selected fragment.

Details

	Subject	Repo	Branch	Lines +/-
	Use jQuery.textSelection API for compat with CodeMirror extension	mediawiki/extensions/WikiEditor	master	+2 -5

Customize query in gerrit

Related Objects

Mentioned In: T255036: When adding link with "link button", link is added on start of page
Mentioned Here: T254705: Show Preview Scrolls to the bottom of the preview (scrolls the edit box into view?)
T254962: Preview scrolls the edit window to the top of the wikitext when using syntax highlighting
rEWED44e82ec08925: Restores focus after dialog closes

Event Timeline

Codas created this task.Jun 5 2020, 5:23 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 5 2020, 5:23 PM

Hi @Codas, thanks for taking the time to report this! Unfortunately this Wikimedia Phabricator task lacks some information.
If you have time and can still reproduce the situation: Please add a more complete description to this task. That should be

a full link to a web address where the issue can be seen.
a clear list of exact steps to reproduce the situation, step by step, so that nobody needs to guess or interpret how you performed each step,
what happens after performing these steps to reproduce,
what you expected to happen instead,
with which web browser(s) and web browser versions you have tested this.

In some situations it can also be helpful to know if you see the same behavior when you are not logged in, your web browser(s) and web browser version(s), your exact MediaWiki version (if it is not on a Wikimedia wiki) and PHP version, etc.

You can edit the task description by clicking Edit Task. Ideally, a good description should allow any other person to follow these steps (without having to interpret steps) and see the same results. Problems that others can reproduce can get fixed faster. Thanks!

@Aklapper before answer I need to ask you where I can find MediaWiki version and PHP version.

@Codas: It says "if it is not on a Wikimedia wiki". Italian Wikivoyage is a Wikimedia wiki.

Firefox 77
Go to https://it.wikivoyage.org/w/index.php?title=Ciampino&action=edit with WikiEditor
Click Evidenziazione della sintassi in editor toolbar to enable syntax highlighting
Highlight Roma in ::per la linea Roma-Albano
Click third item (Collegamento) in editor toolbar to add link
Click Inserisci collegemento button in dialog
Line becomes ::per la linea [[Roma]]-Albano, as expected.

Cannot reproduce. Please provide a complete list of steps, step by step, if your results are different.

I can verify this bug. It's caused by MediaWiki-extensions-CodeMirror. I believe this is the same thing as what @kaldari is talking about here. on English Wikipedia.

Basically what happens is that if you've enabled syntax highlighting, clicking any of the toolbar item may insert the item at the beginning of the textarea instead of where your cursor is. Also (at least for me) it does not happen consistently. So sometimes it works (place item where your cursor is) and sometimes it doesn't ( place item at the beginning, disregarding cursor location)

Aklapper renamed this task from Bug with link and image buttons to Links and images added at top instead of position in WikiEditor when syntax highlighting is enabled.Jun 6 2020, 9:25 AM

Aklapper added projects: MediaWiki-extensions-CodeMirror, WikiEditor.

Restricted Application added a project: Community-Tech. · View Herald TranscriptJun 6 2020, 9:25 AM

Aklapper changed the task status from Stalled to Open.Jun 6 2020, 9:25 AM

Confirmed on frwiki too. Here an example.

Basically happen with Chrome (last version) but not with Safari 13.1.1. Nothing change if I'm logged or unlogged.

Hmm, this error is reproduced on all wikis. Chrome 83.0.4103.97.

Iniquity moved this task from Backlog to Bugs on the MediaWiki-extensions-CodeMirror board.Jun 8 2020, 1:18 PM

Iniquity changed the subtype of this task from "Task" to "Bug Report".

Iniquity updated the task description. (Show Details)Jun 8 2020, 1:28 PM

Aklapper renamed this task from Links and images added at top instead of position in WikiEditor when syntax highlighting is enabled to Links and images added at top instead of position in WikiEditor when syntax highlighting is enabled (on Chrome only).Jun 8 2020, 1:42 PM

Aklapper added a project: Browser-Support-Google-Chrome.

kaldari renamed this task from Links and images added at top instead of position in WikiEditor when syntax highlighting is enabled (on Chrome only) to Links, images, tables, and citations added at top instead of position in WikiEditor when syntax highlighting is enabled (on Chrome only).Jun 8 2020, 3:04 PM

kaldari triaged this task as High priority.

kaldari merged a task: T254626: Inserting links, images, tables, or citations in the WikiText editor in Chrome places them at the beginning.

This seems to be caused by rEWED44e82ec08925: Restores focus after dialog closes, but I don't know why.

(Actually, I'm quite surprised that this worked correctly before, since the code there appears to handle just a plain textarea and not any of the alternative editors.)

Change 603558 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/WikiEditor@master] Use jQuery.textSelection API for compat with CodeMirror extension

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

gerritbot added a project: Patch-For-Review.Jun 8 2020, 5:55 PM

Change 603558 merged by jenkins-bot:
[mediawiki/extensions/WikiEditor@master] Use jQuery.textSelection API for compat with CodeMirror extension

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

ReleaseTaggerBot added a project: MW-1.35-notes (1.35.0-wmf.36; 2020-06-09).Jun 9 2020, 2:00 PM

Maintenance_bot removed a project: Patch-For-Review.Jun 9 2020, 2:10 PM

kaldari closed this task as Resolved.Jun 9 2020, 4:27 PM

kaldari claimed this task.

Aklapper mentioned this in T255036: When adding link with "link button", link is added on start of page.Jun 10 2020, 3:52 PM

Janbery merged a task: T255036: When adding link with "link button", link is added on start of page.Jun 10 2020, 3:59 PM

Janbery subscribed.