-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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.
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
Add spaceless filter #2872
Add spaceless filter #2872
Conversation
This PR was squashed before being merged into the 1.x branch (closes #2872). Discussion ---------- Add spaceless filter Commits ------- a22a5c4 fixed CS 743767b re-implemented the spaceless tag to reuse the filter tag logic b823898 added a spaceless filter 1ba7941 removed unneeded usage of spaceless in tests
I suggest adding a note in the doc about the performance impact of this. Spaceless happens at runtime, and calling it a lot can hurt performance. |
@fabpot What's the reason for deprecating the tag? Removing code duplication or something else? |
Removing duplication, and also advocating for the right usage of extension points. Modifying a string to remove spaces is what a filter should be doing, not a tag. |
Thanks @stof ! |
No description provided.