-
Notifications
You must be signed in to change notification settings - Fork 235
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
false positive on no-whitespace-for-layout #2899
Comments
Currently, as the rule doc says, this rule checks any text nodes for excess whitespace, including text inside or outside attributes. But I think it's a fair question to ask if the rule should be checking the values of arbitrary attributes. Arguably, the I didn't immediately see a justification for this behavior in the original rule PR #819. Maybe @MelSumner has some context on whether this is intended behavior. |
@bmish I'd be okay with excluding attribute values from these checks, that seems reasonable. |
I'm getting a linting error on the following code (running ember-template-lint 5.10.0):
it seems to be angry about the whitespace after the inline handlebars in the class attribute (column 142 of that line). I know this isn't the prettiest code, but shouldn't the rule ignore attributes?
The text was updated successfully, but these errors were encountered: