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

vim: Add a setting to change the Vim mode indicator position on the status bar from right to left #22048

Closed

Conversation

helgemahrt
Copy link
Contributor

Add the first request on issue #14093

Release Notes:

  • Added a setting to change the Vim mode indicator position on the status bar from right to left

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Dec 15, 2024
@maxdeviant maxdeviant changed the title Vim: Added a setting to change the Vim mode indicator position on the status bar from right to left vim: Add a setting to change the Vim mode indicator position on the status bar from right to left Dec 15, 2024
@helgemahrt helgemahrt force-pushed the vim-mode-indicator-setting branch from dd23300 to 48ac070 Compare December 16, 2024 09:02
@helgemahrt
Copy link
Contributor Author

@maxdeviant better now?

@ConradIrwin
Copy link
Member

@helgemahrt Thanks for this!

We should put the mode indicator to the left of the diagnostics, otherwise when the diagnostics update the mode indicator jumps around. The tricky bit is that the diagnostics indicator currently provides the vertical line, so we'll need to to fix how that is rendered.

The only reason this is on the right today is this issue, so I'd also be in favor of just moving this to the left unconditionally and saving ourselves a probably unnecessary setting.

@ConradIrwin ConradIrwin self-assigned this Dec 18, 2024
@helgemahrt helgemahrt force-pushed the vim-mode-indicator-setting branch from 1df35fe to 5ad3ada Compare December 18, 2024 21:17
@helgemahrt
Copy link
Contributor Author

IMO it's probably a good idea to have it configurable - there may be people who prefer it on the right side. However, maybe the better approach would be to make the order of all elements of the status bar configurable?

For now, I added a change that ensures that the diagnostics indicator stays the right-most element on the left side of the status bar.

@helgemahrt
Copy link
Contributor Author

Hang on, I made a mess in this branch when I removed the other commits. I'll create a clean branch/PR and link it here.

@ConradIrwin
Copy link
Member

@helgemahrt I think it needs to be the left-most position on the left of the status bar. I don't want it to move when the diagnostics updates.

@helgemahrt helgemahrt force-pushed the vim-mode-indicator-setting branch from 5ad3ada to fb4a6ca Compare December 18, 2024 21:33
@helgemahrt
Copy link
Contributor Author

I think I managed to recover this branch. 😅

@helgemahrt I think it needs to be the left-most position on the left of the status bar. I don't want it to move when the diagnostics updates.

Currently, the diagnostics indicator is the right-most item on the left side, with the vertical separator to the left:
image

My latest change ensures it stays that way, but, TBH, it feels a bit hacky.
image

I just realized I missed ensuring that the Vim mode indicator stays behind the diagnostics indicator on startup as well. Will fix that in a moment.

@helgemahrt
Copy link
Contributor Author

I just realized that there's also the activity indicator. Now I'm thinking that maybe it'd be better to scrap this PR and rethink whether and how the status bar should be configurable. And maybe put the vim mode indicator to the left side for the time being, like you suggested. WDYT?

@ConradIrwin
Copy link
Member

Sounds good! If you're up for it I think moving to the left of all the things would be a positive development

@helgemahrt
Copy link
Contributor Author

Created #22269 to move the indicator. Closing this PR.

@helgemahrt helgemahrt closed this Dec 19, 2024
@helgemahrt helgemahrt deleted the vim-mode-indicator-setting branch December 19, 2024 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants