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.

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

Alignment indentation fails when more than two tree nodes are nested in the same row. #8650

Closed
gh16980973 opened this issue Oct 28, 2023 · 1 comment · Fixed by #8659
Closed
Labels
A-indent Area: Indentation C-bug Category: This is a bug

Comments

@gh16980973
Copy link

gh16980973 commented Oct 28, 2023

Summary

; The resulting indentation:

(let ((a (begin a
        |))
 a)

; Expected format:

(let ((a (begin a
               |))
 a)

Reproduction Steps

; indents.scm
(list . (symbol) @item @anchor (#eq? @item "let")) @align
(list . (symbol) @item (_) @anchor (#not-eq? @item "let")) @align

Platform

Ubuntu 22.04 (WSL)

Terminal Emulator

Windows Terminal

Helix Version

helix 23.10 (f992c3b)

@gh16980973 gh16980973 added the C-bug Category: This is a bug label Oct 28, 2023
@pascalkuthe pascalkuthe added the A-indent Area: Indentation label Oct 28, 2023
@Triton171
Copy link
Contributor

Nice catch! Until now, there haven't really been any situations where multiple @align captures occur on the same line, so this bug was never triggered. #8659 should fix the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-indent Area: Indentation C-bug Category: This is a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants