Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Syntax highlight broken if text contains ampersand #181

Closed
karlschwaier opened this issue Dec 13, 2017 · 4 comments · Fixed by #183
Closed

Syntax highlight broken if text contains ampersand #181

karlschwaier opened this issue Dec 13, 2017 · 4 comments · Fixed by #183

Comments

@karlschwaier
Copy link

karlschwaier commented Dec 13, 2017

Description

If a string contains an ampersand, the syntax highlight after that ampersand is broken.

bildschirmfoto 2017-12-13 um 15 03 02

Versions

Atom 1.23.0
language-html 0.48.4

Additional Information

Similar to #141.

@webstaurantstore
Copy link

<iframe height="500px" width="500px"
  src="/v/?num=&height=&width=&autostart=true&repeat=false&force_dimensions=true&controlbar=false&disable_pause=true&disable_opening_frame=true&wmode=transparent&presentation_mode=product_preview&allow_full_screen=false"
  frameborder="0" allowFullScreen="false" webkitallowfullscreen="false" mozallowfullscreen="false"></iframe>

Also happens with this string, but only after the 4th ampersand.
Atom 1.22.1
language-html 0.48.2

@jwardbond
Copy link

It happens when an ampersand is immediately followed by any alphabetical character.

image

vs.

image

Atom 1.23.1
language-html 0.48.4

@50Wliu
Copy link
Contributor

50Wliu commented Dec 20, 2017

@jonathanfleckenstein your issue should be fixed in Atom 1.23.0. I'm looking into the other issues now.

@webstaurantstore
Copy link

@50Wliu Thank you, hope you're having a great Holiday!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants