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.

Sign up for GitHub

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

Jump to bottom

Formatting deletes document's body when there's a DTD declaration #198

Closed
fbricon opened this issue Nov 6, 2018 · 0 comments
Closed

Formatting deletes document's body when there's a DTD declaration #198

fbricon opened this issue Nov 6, 2018 · 0 comments
Assignees
@NikolasKomonen
Labels
bug Something isn't working DTD formatting This issue or enhancement is related to formatting support
Milestone
v0.0.2

Comments

@fbricon
Copy link
Contributor

fbricon commented Nov 6, 2018

Formatting this:

<?xml version="1.0"?>
<!DOCTYPE student [
  <!ELEMENT student (surname,id)>
  <!ELEMENT surname (#PCDATA)>
  <!ELEMENT id (#PCDATA)>
]>
<student>
  <surname>Smith</surname>
  <id>567896</id>
</student>

yields

<?xml version="1.0"?>
<!DOCTYPE student [
  >
@fbricon fbricon added bug Something isn't working formatting This issue or enhancement is related to formatting support labels Nov 6, 2018
@fbricon fbricon added this to the v0.0.2 milestone Nov 6, 2018
NikolasKomonen pushed a commit to NikolasKomonen/lsp4xml that referenced this issue Nov 6, 2018
@angelozerr @NikolasKomonen
DTD losing content on format fixed
63116b7 
Fixes eclipse-lemminx#198
@NikolasKomonen NikolasKomonen mentioned this issue Nov 6, 2018
Merged
NikolasKomonen pushed a commit to NikolasKomonen/lsp4xml that referenced this issue Nov 6, 2018
@angelozerr @NikolasKomonen
DTD losing content on format fixed
a8c1f9f 
Fixes eclipse-lemminx#198
NikolasKomonen pushed a commit to NikolasKomonen/lsp4xml that referenced this issue Nov 7, 2018
@angelozerr @NikolasKomonen
DTD losing content on format fixed
379e05f 
Fixes eclipse-lemminx#198
fbricon pushed a commit that referenced this issue Nov 7, 2018
@angelozerr @fbricon
DTD losing content on format fixed
ca150a3 
Fixes #198
@fbricon fbricon added the DTD label Nov 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NikolasKomonen NikolasKomonen

Labels
bug Something isn't working DTD formatting This issue or enhancement is related to formatting support
Projects
None yet
Milestone
v0.0.2
Development

No branches or pull requests
2 participants
@fbricon @NikolasKomonen