Page MenuHomePhabricator

The first opening bracket of a tvar starting a line misses cm-mw-tag-translate class
Open, Needs TriagePublicBUG REPORT

Description

Follow-up of T284883: CodeMirror – support highlighting the new <tvar> tag syntax.

Steps to replicate the issue:

Open the 2010 Wikitext Editor, CodeMirror enabled, and type the following content:

<translate>
Hello <tvar name=world>world</tvar>

<tvar name=foo>Foo</tvar>
</translate>

What happens?:
Note the first opening bracket < for foo tvar is not green-colored, whereas the one of world tvar actually is.
When the tvar is at line start, the cm-mw-tag-translate class is not added to the leading bracket.

What should have happened instead?:
The bracket should be colored as world bracket, with cm-mw-tag-translate class.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Bhsd moved this task from Backlog to Done on the MediaWiki-extensions-CodeMirror board.
Bhsd subscribed.

This bug does not exist any more.

I cannot reproduce at mediawiki.org, but I can still reproduce at meta.wikimedia.org, even in private browsing with &safemode=1.

At MetaWiki (issuing behavior):

<span role="presentation">
<span class=" cm-mw-exttag-bracket cm-mw-ext-tvar">&lt;</span>
<span class="cm-mw-tag-translate cm-mw-exttag-name cm-mw-ext-tvar">tvar </span>
<span class="cm-mw-tag-translate cm-mw-exttag-attribute cm-mw-ext-tvar">name=foo</span>
<span class="cm-mw-tag-translate cm-mw-exttag-bracket cm-mw-ext-tvar">&gt;</span>
<span class="cm-mw-tag-translate cm-mw-tag-tvar ">Foo</span>
<span class="cm-mw-tag-translate cm-mw-exttag-bracket cm-mw-ext-tvar">&lt;/</span>
<span class="cm-mw-tag-translate cm-mw-exttag-name cm-mw-ext-tvar">tvar</span>
<span class="cm-mw-tag-translate cm-mw-exttag-bracket cm-mw-ext-tvar">&gt;</span>
</span>

At MediaWiki.org (expected behavior):

<div class="cm-line">
<span class="cm-mw-tag-translate cm-mw-exttag-bracket cm-mw-ext-tvar">&lt;</span>
<span class="cm-mw-tag-translate cm-mw-exttag-name cm-mw-ext-tvar">tvar </span>
<span class="cm-mw-tag-translate cm-mw-exttag-attribute cm-mw-ext-tvar">name=foo</span>
<span class="cm-mw-tag-translate cm-mw-exttag-bracket cm-mw-ext-tvar">&gt;</span>
<span class="cm-mw-tag-translate cm-mw-tag-tvar">Foo</span>
<span class="cm-mw-tag-translate cm-mw-exttag-bracket cm-mw-ext-tvar">&lt;/</span>
<span class="cm-mw-tag-translate cm-mw-exttag-name cm-mw-ext-tvar">tvar</span>
<span class="cm-mw-tag-translate cm-mw-exttag-bracket cm-mw-ext-tvar">&gt;</span>
</div>

We notice the cm-mw-tag-translate class is missing from enclosing span.