-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Fix incorrect gutter bail message #7534
Conversation
Please run rustfmt to make CI pass |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change in the bail!
message looks good but I believe the configuration docs are correct:
helix/helix-view/src/editor.rs
Lines 86 to 99 in 57e538d
impl Default for GutterConfig { | |
fn default() -> Self { | |
Self { | |
layout: vec![ | |
GutterType::Diagnostics, | |
GutterType::Spacer, | |
GutterType::LineNumbers, | |
GutterType::Spacer, | |
GutterType::Diff, | |
], | |
line_numbers: GutterLineNumbersConfig::default(), | |
} | |
} | |
} |
Gutter elements are repeatable and spacer is used to add some space between other elements.
ah right I misserad the change. The docs show the default configureitaton (which contains two spacers) not the list of available options |
"spacer"
was listed twice in theeditors.gutter
example and the error message for the gutter config parsing was missing newer options.