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 ResizePane scale_factor arg #23844

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

universalmind303
Copy link

@universalmind303 universalmind303 commented Jan 29, 2025

Closes #ISSUE
#23298
Release Notes:

Improved vim::ResizePane by adding optional scale_factor argument to increase or decrease the amount the pane is resized

Copy link

cla-bot bot commented Jan 29, 2025

We require contributors to sign our Contributor License Agreement, and we don't have @universalmind303 on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@universalmind303
Copy link
Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jan 29, 2025
Copy link

cla-bot bot commented Jan 29, 2025

The cla-bot has been summoned, and re-checked this pull request!

@maxdeviant maxdeviant changed the title add resizePane scale_factor arg vim: Add resizePane scale_factor arg Jan 29, 2025
@maxdeviant maxdeviant changed the title vim: Add resizePane scale_factor arg vim: Add ResizePane scale_factor arg Jan 29, 2025
@ConradIrwin
Copy link
Member

@universalmind303 Thanks for this.

I'm curious if you want to have different keybindings with different scale factors, or if this would be easiest if it was a setting instead. (We might even stray from vim by making it more than 1 by default - it's very slow to resize panes column by column on large monitors).

The keybinding approach could work, but it adds complexity to an already complex thing.

@ConradIrwin ConradIrwin self-assigned this Jan 31, 2025
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.

2 participants