-
Notifications
You must be signed in to change notification settings - Fork 67
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
Add masterstack layout #1556
base: master
Are you sure you want to change the base?
Add masterstack layout #1556
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1556 +/- ##
========================================
- Coverage 95.3% 95.1% -0.3%
========================================
Files 133 133
Lines 12054 12089 +35
========================================
Hits 11499 11499
- Misses 555 590 +35 |
I was reluctant to add master stack because a frame in masterstack layout is equivalent to hc load '(split horizontal:0.6:0 (clients vertical:0) (clients vertical:0 ))' (I see that 60% is hardcoded in your PR #1556). I fear that we're duplicating the behaviour of manual tiling because with masterstack implemented, how do you adjust the ratio of 60%?
I found the bottom one surprising when testing your code. Does the middle one make sense? By the way, the above mentioned manual split simply remembers the node when focusing from left to right. On the other hand I understand that it's too many keystrokes to split/unsplit manually all the time when switching between one and two clients. Also, I see that many people ask for masterstack so probably I won't object. mergin this pr when it is more mature! |
3f53af6
to
0bfb8b2
Compare
Didn't answer last year for some reason, sorry.
The reason I used the bottom one at that time was that it is the most recent node. Later I switched that to the top node because thats easier for your eyes. Middle would also be a valid choice, but I feel like then it's hard to see which one is the middle one (e.g. when there are 4 nodes in the stack). Best case would probably be remembering the node like it would be for the manual case, but I guess that complicates the code quite a bit. I have used the current version of the code for the past year, so I would say functionality wise it's solid already. Maybe I can find some time to finish the PR if people are interested in adding the layout upstream. Otherwise I'll just keep using it like it currently is. |
This PR adds the master stack layout as discussed in #1003.
TODOs: