Page MenuHomePhabricator

Syntax highlighting in 2017 wikitext editor has extreme vertical cursor displacement
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Open any article in source mode using the 2017 wikitext editor with syntax highlighting enabled.

What happens?:
Highlight cursor / selections appear massively offset from the underlying text.

image.png (818×1 px, 196 KB)

What should have happened instead?:
Cursor / highlight should match the position of the underlying text.

Event Timeline

Change 992760 had a related patch set uploaded (by DLynch; author: DLynch):

[VisualEditor/VisualEditor@master] Zero out padding in source mode paragraphs as well

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

taavi triaged this task as Unbreak Now! priority.Jan 24 2024, 7:39 PM

Change 992774 had a related patch set uploaded (by Jdrewniak; author: Jdrewniak):

[mediawiki/skins/Vector@master] Revert "Update <p> spacing to improve consistency of ul/ol spacing, also update heading spacing to be more consistent, relying on mw defaults more"

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

Change 992774 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Revert "Update <p> spacing to improve consistency of ul/ol spacing, also update heading spacing to be more consistent, relying on mw defaults more"

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

Change 992775 had a related patch set uploaded (by Jforrester; author: Jdrewniak):

[mediawiki/skins/Vector@wmf/1.42.0-wmf.15] Revert "Update <p> spacing to improve consistency of ul/ol spacing, also update heading spacing to be more consistent, relying on mw defaults more"

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

Change 992775 merged by jenkins-bot:

[mediawiki/skins/Vector@wmf/1.42.0-wmf.15] Revert "Update <p> spacing to improve consistency of ul/ol spacing, also update heading spacing to be more consistent, relying on mw defaults more"

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

Mentioned in SAL (#wikimedia-operations) [2024-01-24T23:19:29Z] <jforrester@deploy2002> Started scap: Backport for [[gerrit:992775|Revert "Update <p> spacing to improve consistency of ul/ol spacing, also update heading spacing to be more consistent, relying on mw defaults more" (T355805 T354433)]]

Mentioned in SAL (#wikimedia-operations) [2024-01-24T23:21:01Z] <jforrester@deploy2002> jforrester: Backport for [[gerrit:992775|Revert "Update <p> spacing to improve consistency of ul/ol spacing, also update heading spacing to be more consistent, relying on mw defaults more" (T355805 T354433)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2024-01-24T23:32:59Z] <jforrester@deploy2002> Finished scap: Backport for [[gerrit:992775|Revert "Update <p> spacing to improve consistency of ul/ol spacing, also update heading spacing to be more consistent, relying on mw defaults more" (T355805 T354433)]] (duration: 13m 29s)

Change 993505 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):

[mediawiki/skins/Vector@master] Update <p> spacing to improve consistency of ul/ol spacing, also update heading spacing to be more consistent, relying on mw defaults more (attempt 2)

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

Change 992760 merged by jenkins-bot:

[VisualEditor/VisualEditor@master] Zero out padding in source mode paragraphs as well

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

Change 997584 had a related patch set uploaded (by Esanders; author: Esanders):

[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (4ec4a75e4)

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

Change 997584 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (4ec4a75e4)

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