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

Can't deal with XHTML5 #828

Closed
garretwilson opened this issue Dec 13, 2015 · 4 comments
Closed

Can't deal with XHTML5 #828

garretwilson opened this issue Dec 13, 2015 · 4 comments
Labels
language: html type: bug
Milestone
v1.6.0

Comments

@garretwilson
Copy link

I'm using XHTML5, which for most practical purposes means HTML5 using the XML syntax---which again is pretty much HTML5 syntax except that it's stricter.

http://www.w3.org/TR/html5/introduction.html#html-vs-xhtml

I have a nice little XHTML5 document, starting with this:

<?xml version="1.0" encoding="utf-8"?>
<!DOCTYPE html>
<html lang="en-us" xmlns="http://www.w3.org/1999/xhtml">

Unfortunately atom-beautify (which uses js-beautify) thinks that the XML declaration is a tag or something, and indents the entire document except for the first line!

<?xml version="1.0" encoding="utf-8"?>
    <!DOCTYPE html>
    <html lang="en-us" xmlns="http://www.w3.org/1999/xhtml">
    ....

Please recognize the XML declaration in XHTML documents, and don't cause the entire documented because of this. Thanks.
@garretwilson garretwilson mentioned this issue Dec 13, 2015
Closed
@bitwiseman bitwiseman added this to the v1.6.0 milestone Dec 13, 2015
@garretwilson
Copy link
Author

@bitwiseman , sincere thanks for accepting my bug report and scheduling it to be fixed. It is a refreshing change from other bug reports for other tools, where they have no idea what XHTML is and just say "we don't support XML".

Any idea when 1.6.0 is scheduled to be released? Until then I may have to learn how to write Atom plugins just to write a workaround for the atom-beautify plugin.

Thanks again.

@bitwiseman
Copy link
Member

Looks like this might be related or partly addressed by #683.
Release date for 1.6.0 is when I get together enough changes to warrant it. Probably sometime in January.

@bitwiseman
Copy link
Member

Fixed!

@garretwilson garretwilson mentioned this issue Mar 17, 2016
Closed
@pyup-bot pyup-bot mentioned this issue Mar 5, 2017
Merged
@garretwilson
Copy link
Author

I just wanted to say thanks. I had stopped using XML declarations for a while to work around this problem, but a few months ago I realized this was working and brought them back. It helps a lot and I appreciate it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
language: html type: bug
Projects
None yet
Milestone
v1.6.0
Development

No branches or pull requests
2 participants
@garretwilson @bitwiseman