Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(xml) line with single letter namespace prefix not highlighted since 10.4.0 #2957

Closed
boghyon opened this issue Jan 11, 2021 · 0 comments · Fixed by #2958
Closed

(xml) line with single letter namespace prefix not highlighted since 10.4.0 #2957

boghyon opened this issue Jan 11, 2021 · 0 comments · Fixed by #2958
Labels
bug help welcome Could use help from community language

Comments

@boghyon
Copy link

boghyon commented Jan 11, 2021

Are you using highlight or highlightAuto?
highlight

Sample Code to Reproduce

<abc:OK xmlns:abc="..." />
<ab:OK xmlns:ab="..." />
<a:FAIL xmlns:a="..." /> <!-- ← No syntax highlighting for you -->
<OK xmlns="..." />

https://jsbin.com/gexeyen/edit?html,js,output

output jsbin com_gexeyen(Moto G4)
output jsbin com_gexeyen(Moto G4) (2)

Expected behavior

<abc:OK xmlns:abc="..." />
<ab:OK xmlns:ab="..." />
<a:OK xmlns:a="..." />
<OK xmlns="..." />
<abc:OK xmlns:abc="..." />
<ab:OK xmlns:ab="..." />
<a:OK xmlns:a="...">
  <AllOK xmlns="..." />
</a:Ok>
<OK xmlns="..." />

Additional context

@boghyon boghyon added bug help welcome Could use help from community language labels Jan 11, 2021
boghyon referenced this issue in SAP/openui5 Jun 3, 2021
Code highlighter used until now in Demo Kit is obsolete and in order to
be in best shape, it is changed with a better alternative.
The new highlighter used for highlighting code blocks in Demo Kit is
called highlighter.js

JIRA: BGSOFUIPIRIN-4840

Change-Id: I901960f72f04d1e19fe4221d8e65037c44a21378
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug help welcome Could use help from community language
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant