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

Always on top feature #829

Open
wants to merge 35 commits into
base: master
Choose a base branch
from
Open

Always on top feature #829

wants to merge 35 commits into from

Conversation

pro470
Copy link

@pro470 pro470 commented May 16, 2024

I create a feature which allows a window to be always on the focused workspace. So you only need to focus the workspace I have some things still to do:

-Clean up
-Window Manager Event on Destroy and Unmanaged

a more detailed description is coming when I got everything done but I wanted to make the pull request now because I promise it in the discord chat.

@pro470 pro470 marked this pull request as ready for review May 16, 2024 17:41
@LGUG2Z LGUG2Z force-pushed the master branch 2 times, most recently from 0c842e4 to 05af7ce Compare May 19, 2024 21:27
@pro470
Copy link
Author

pro470 commented May 20, 2024

I finished all the work I have to do so this is a PR to enable the feature to have a window always on top so when you change the workspace the always-on-top window will move with the focus workspace. This is helpful if you have multiple monitors and are focusing on a specific workspace but also want to move a window. This feature will shorten the command you have to run to achieve this. There are also different scenarios in which this command can also be handy like moving multiple windows at once.

This is my first PR so if something is wrong or you want me to change something or give me criticism PLEASE DO. I hope this PR can be helpful for somebody

pro470 and others added 3 commits May 25, 2024 13:40
This commit ensures that the "send" behaviour is respected in
named-workspace command variants such as send-to-named-workspace.
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.

None yet

2 participants