Page MenuHomePhabricator

CodeMirror syntax highlighting shows overlapping text when word-wrap (hyphens, word-break) is overridden
Closed, ResolvedPublic

Description

Someone has reported (in T184467#4124483 and T184467#4105987 and elsewhere in that ticket) that in the 2017 Wikitext editor with CodeMirror enabled the text will display as fuzzy. This appears to be because word-wrap, hyphens, or word-break is overridden in the users's browser.

lorem-ipsum-screenshot-180404 Kopie.png (456×879 px, 283 KB)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Looking at the screenshot, the problem isn't with word-wrap, but with [[https://developer.mozilla.org/en-US/docs/Web/CSS/hyphens | hyphens]]. So the value should be reset to manual or none, to make it match the overlay (though setting this to auto on editable content seems bad anyway).

Here's another example, from today. I want to challenge the assignment of 'low' priority to this bug. I encounter this error more than half of the time when I edit in Firefox. I'm disabling the 2017 WTE until it's fixed; it basically prevents me from using the feature in my chosen browser, and it's not an edge case. @TBolliger can you give me a sense of why this is being de-prioritized?

Screenshot-2018-4-12 Editing Talk Wikimedia Foundation Annual Plan 2018-2019 Draft - Meta(1).png (144×2 px, 71 KB)

TBolliger renamed this task from CodeMirror syntax highlighting shows overlapping text when word-wrap (word-break) is overridden to CodeMirror syntax highlighting shows overlapping text when word-wrap (hyphens, word-break) is overridden.Apr 12 2018, 7:51 PM
TBolliger raised the priority of this task from Low to Medium.
TBolliger updated the task description. (Show Details)
TBolliger added a subscriber: Esanders.

We only received one report of this hyphenation defect and were not able to reproduce on the same browser/OS combination, thus the lower priority.

@Esanders — Any thoughts on @Schnark's comments in T192019#4126302 ?

matmarex added a subscriber: matmarex.

Seems like an easy fix, similar to this commit we merged today: https://gerrit.wikimedia.org/r/#/c/422346/

Change 426949 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/CodeMirror@master] Merge selectors for VE & CM surfaces, and add 'hyphens' rule

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

Change 426949 merged by jenkins-bot:
[mediawiki/extensions/CodeMirror@master] Merge selectors for VE & CM surfaces, and add 'hyphens' rule

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

@matmarex / @Esanders Could one of you look into this? Thank you.

@Aschmidt reported that it is still not fixed for him.

Based on the screenshot in https://de.wikipedia.org/w/index.php?title=Benutzer_Diskussion:Aschmidt&oldid=178336976#Technische_Wünsche:_Wikitext-Syntaxhervorhebung_ist_jetzt_Standardfunktion, that seems like a different issue:

image.png (78×246 px, 9 KB)

There is no word-wrapping in the screenshot, and that was the cause of this problem.

@Aschmidt If you still see that problem, can you file another task? (yet another…)