Removed newlines from asset templates #36

Open
wants to merge 1 commit into
from

Projects

None yet

3 participants

@tbjers
tbjers commented Aug 12, 2013

Newlines are not needed here since the Liquid tag will have a newline after it. Previous template breaks javascript variables if the user wants to include the asset tag inside a variable for programmatic inclusion on a specific condition. Seems silly that the user should have to override the templates just for this.

@tbjers tbjers Removed newlines from asset templates
Newlines are not needed here since the Liquid tag will have a newline
after it.
ce968ab
@wei2912
wei2912 commented Oct 4, 2013

Tests are broken because they're testing for a newline. You will need to update the tests as well. :)

@kitsched

Integrated into my fork here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment