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

Highlighting embedded XML in HEREDOC block #199

Merged
merged 1 commit into from
Jun 13, 2017
Merged

Conversation

deepj
Copy link
Contributor

@deepj deepj commented May 5, 2017

Description of the Change

It enables highlighting embedded XML in HEREDOC block. I personally work with a lot of XML nowadays so I hope someone else than me can benefit of that as well 🙃

Alternate Designs

No

Benefits

It's possible to see XML syntax in HEREDOC block

Possible Drawbacks

Unknown

Applicable Issues

Unknown

@50Wliu 50Wliu self-assigned this May 5, 2017
'beginCaptures':
'0':
'name': 'punctuation.definition.string.begin.ruby'
'contentName': 'source.xml'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be text.xml instead of source.xml, same on line 1468.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why? Is there any difference between text.xml and source.xml here?

Copy link
Contributor

@50Wliu 50Wliu Jun 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The scopeName of language-xml is text.xml.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I see. I'll fix it asap

@deepj
Copy link
Contributor Author

deepj commented Jun 13, 2017

@50Wliu fixed
@infininight thx for the help

@50Wliu 50Wliu merged commit 8fc7b87 into atom:master Jun 13, 2017
@50Wliu
Copy link
Contributor

50Wliu commented Jun 13, 2017

Thanks!

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

Successfully merging this pull request may close these issues.

3 participants