-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
add spacer element to statusline #3165
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
Conversation
This would probably need to be added to the documentation ( |
Of course. Sorry for the oversight. Will update. |
Ready for review. |
Currently, gutters uses |
@@ -246,6 +246,9 @@ pub enum StatusLineElement { | |||
|
|||
/// The cursor position | |||
Position, | |||
|
|||
/// A single space | |||
Spacer, |
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.
GutterType
uses Padding
terminology.
helix-view/src/editor.rs
on line 330.
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.
Gutters should also use the spacer terminology, I'll rename there
@@ -78,6 +78,7 @@ The following elements can be configured: | |||
| `diagnostics` | The number of warnings and/or errors | | |||
| `selections` | The number of active selections | | |||
| `position` | The cursor position | | |||
| `spacer` | Inserts a space between elements (multiple/contiguous spacers may be specified) | |
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.
Gutters uses "padding"
.
/book/src/configuration.md
on line 41.
| `gutters` | Gutters to display: Available are `diagnostics` and `line-numbers` and `padding`, note that `diagnostics` also includes other features like breakpoints | `["diagnostics", "line-numbers", "padding"]` |
To me, "padding" implies inserting spaces at the front (or back) of strings to align them to columns. That's not what these do, but I'm happy to change the name if there's a consensus. c.f.. #3106 (comment) |
* add spacer element to statusline * docs
Addresses #3106 and associated other comments. Multiple
"spacer"
elements may be specified in each statusline section.