bismuth icon indicating copy to clipboard operation
bismuth copied to clipboard

feat: smartgaps: Hide gaps if showing a single window

Open defer opened this issue 2 years ago • 3 comments

Summary

Adds a toggle that will hide gaps if only a single window is visible. The feature is disabled by default.

Implementation added to both the cpp and ts engines.

Breaking Changes

No

UI Changes

Before After
before after

Test Plan

  1. Make sure gaps are enabled, enable the smart gaps toggle, apply.
  2. Put a single window in the workspace, observe that there are no gaps.
  3. Open a second window, observe that there are gaps.

Related Issues

Closes #404

defer avatar Aug 05 '22 12:08 defer

I had to bypass precommit hooks because clang-format is complaining about existing code.

defer avatar Aug 05 '22 12:08 defer

Wait, but we already have a such feature:

image

What is the difference?


Yeah, the issue is actually closed https://github.com/Bismuth-Forge/bismuth/issues/404#issuecomment-1207200369

HanabishiRecca avatar Sep 16 '22 11:09 HanabishiRecca

@defer So do you think maybe we can close this PR now as un-merged perhaps?

Also (BTW) when i enable this option: "Maximize sole window" it does seem to help or partially fix some other window bug? Whereby the tiling is partially broken / semi-works. Depending on certain windows / some of the virtual desktops / virtual screens. It's all a bit new to me still.

Anyhow if we can know to actual close this PR (that it does not add anything new / or better / worthwhile improvements needed). Then many thanks

dreamcat4 avatar Jun 07 '23 14:06 dreamcat4