Skip to content

Multiline indented #292

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Multiline indented #292

wants to merge 2 commits into from

Conversation

wkillerud
Copy link
Owner

Opening a draft for visibility. With Sass 1.84.0 the indented syntax got a new behavior for newlines.

line breaks only end statements when a statement can end, and in any other case, a line break is treated as non-meaningful white space.

The parser in Some Sass needs work to adapt to this change.

If you are eager to use this language feature, help would be very much appreciated. The PR adds two tests (debuggable) that show some of the possibilities with this new behavior, but this affects pretty much the whole parser.

In the indented syntax, authors can use a semicolon to explicitly end a statement. Subsequent lines in the same block still must have the same level of indentation. This means that the indented format won't support multiple statements on a single line, even if they are separated by a semicolon.

Solved in 1f89326

They are allowed to explicitly end statements in Sass 1.84.0
@wkillerud wkillerud added the help wanted Extra attention is needed label Mar 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant