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

Excessive newlines in output #50

Closed
chrylis opened this issue Jun 27, 2014 · 2 comments
Closed

Excessive newlines in output #50

chrylis opened this issue Jun 27, 2014 · 2 comments
Assignees
@ultraq
Labels
enhancement

Comments

@chrylis
Copy link

chrylis commented Jun 27, 2014

The layout processor appears to include all of the newlines present in both source files, even as it eliminates the tags that are made redundant by whatever operations. This leads to rendered output with blank lines scattered at random, making reading the output rather difficult. Trivial issue, but it would be nice if lines made blank by the layout templating were removed entirely.
@ultraq ultraq self-assigned this Jun 29, 2014
@ultraq
Copy link
Owner

ultraq commented Jun 29, 2014

There are a few newlines that used to annoy me in the beginning, but I can't remember what they were there for any more (I think it was an IE conditional comments thing). If those are no longer necessary, then I'll see what I can do to make the decoration process run without making extra newlines.

ultraq added a commit that referenced this issue Aug 4, 2014
@ultraq
Fix for #50, reduce the amount of whitespace copied around the HTML tag
f3f6d6f
@ultraq
Copy link
Owner

ultraq commented Aug 4, 2014

I could only reduce the amount of whitespace copied surrounding the <html> tag as there was a test to ensure that items like comments outside the <html> were retained, brought about by the fix for #10.

@ultraq ultraq closed this as completed Aug 4, 2014
ultraq added a commit that referenced this issue Aug 4, 2014
@ultraq
Fix for #50, always allow next text element after a comment
10a728e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ultraq ultraq

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ultraq @chrylis