-
-
Notifications
You must be signed in to change notification settings - Fork 32
feat: scrolling layout #91
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
Conversation
|
I like this new scroll layout! I know it's still a WIP, but here are some notes:
|
|
sounds good. the two finger thing is because we are also listening to the scroll event (think mouse wheel) but i think i will forgo that. as for move window to prev/next workspace i would say that you can trivially script that as of now. rifts cli is pretty fast (if not very fast) so it should work quite well. there also are already commands to fullscreen (full width) a window, i just have to implement them in the scrolling layout(theyre a part of our layout engine trait). |
|
It wasn't trivial for me 😄 , but I did get a script to move a window to the next workspace. It runs fast. #!/bin/bash
WS=$(rift-cli query workspaces | jq -r ".[] | select(.is_active == true) | .index")
WS=$((WS + 1))
rift-cli execute workspace move-window $WS
rift-cli execute workspace switch $WSI used this in Keyboard Cowboy with keyboard shortcuts. Thanks for the pointer. |
17b2763 to
a1f33d8
Compare
|
merge conflicts from hell |
|
@tshort let me know how this feels. this branch has also been rebased to the latest main so keep that in mind as there have been pretty significant changes in that time. |
|
Thanks for working on this! I was just playing around with it. Some comments:
|
these should be addressed |
|
Wow, you fixed that faster than the time it took me to write my last response 😄. The only issue I saw is that if I increase the size of the first (left-most) window and I scroll to the right and then scroll back to the left, part of the focused window is hidden. It looks like it's scrolling based on the size of the second window, not the amount necessary to completely fit the first window. |
|
all issues should be resolved (i think). im honestly not testing that much since i don't really like using a scrolling layout so hopefully im not missing anything. |
|
I still see the issue I described above with the wider window. Also, if I'm focused on the larger left-most window, and I repeatedly hit the "focus left" button, the window repeatedly shifts between aligning to the left edge of the screen and a shifted left position. |
|
Your latest commit fixed the scrolling issue for the first and last window. I'm seeing a weird behavior where the mouse cursor acts like the shift key is pressed (I get selections). If I stop rift, it goes away. This happened before your latest commit, too. |
|
I don't see the funny cursor issue on the main branch. |
|
I see that toggling full screen now works in the scrolling layout! With that, I did notice a couple of things. When in full screen, it messes up moving focus left or right--I think it should not allow a focus shift. Another niggle is if I'm on a full-screen window and I switch to another space and switch back, the full-screen window is visible, but the focused window is off by one. Oh wait--now it works right. Either I imagined that, or it only sometimes happens. |
|
okay this should be all good. i could not reproduce the cursor issue so that will need some investigation. so far it seems like you are my source of truth @tshort so let me know if you encounter any issues |
|
Here's more info on the cursor issue:
With trackpad scrolling disabled, the scroll layout works great for me! All the issues I mentioned above are fixed. |
|
@acsandmann, what does your latest commit do? The reason I ask is that I was working on some changes to make scrolling more natural. See https://github.com/tshort/rift/tree/ts-scroll. In that, I made window widths a portion of the screen width. I think you may have started out that way. I did this to change scrolling as follows:
|
|
Another thing I noticed is that new windows don't appear consistently in the right place if |
|
closing in favor of #146 |
supercedes #90