Skip to content

fix: :belowright hor 10Oil#691

Merged
stevearc merged 1 commit intostevearc:masterfrom
phanen:smods
Nov 30, 2025
Merged

fix: :belowright hor 10Oil#691
stevearc merged 1 commit intostevearc:masterfrom
phanen:smods

Conversation

@phanen
Copy link
Contributor

@phanen phanen commented Nov 25, 2025

  • Support more command mods :belowright hor 10Oil
  • Fix :tab Oil only work on the first tab

Extract inline function to make the diff small after format.

* Support command mods `:belowright hor 10Oil`
* Fix `:tab Oil` only work on the first tab
@github-actions github-actions bot requested a review from stevearc November 25, 2025 04:53
@stevearc
Copy link
Owner

stevearc commented Nov 30, 2025

LGTM thanks!

@stevearc stevearc merged commit cbcb3f9 into stevearc:master Nov 30, 2025
10 checks passed
@looshch
Copy link

looshch commented Dec 9, 2025

@phanen @stevearc this PR introduced a regression for vertical Oil

before it was opening equally sized vertical split and spawning an instance of oil there, now it creates a split with a minimum width which shouldn’t be the case

before

before.mov

after

after.mov

@phanen
Copy link
Contributor Author

phanen commented Dec 9, 2025

@looshch fixed in #695.

@looshch
Copy link

looshch commented Dec 9, 2025

lovely @phanen, thanks a ton! Have a great day!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants