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 199 commits into
base: master
Choose a base branch
from
Open

Always on top feature #829

wants to merge 199 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.
@LGUG2Z LGUG2Z force-pushed the master branch 2 times, most recently from fb92ba4 to 692da90 Compare December 26, 2024 22:16
pro470 and others added 10 commits January 3, 2025 01:51
This commit adds an optional --komorebi-config flag to the check command
to allow users to check a komorebi.json file in an arbitrary location.
This commit adds support for specifying a layout flip axis for each
workspace in the static configuration file.
This commit adds mutex lock scoping in
WindowManager::enforce_workspace_rule to avoid a deadlock when
should_update_focused_workspace evaluates to true.

fix LGUG2Z#1212
If a user triggers the workspace reconciliator by clicking on an app in
the start bar or via alt-tab, a notification should be sent to
subscribers such as komorebi-bar so that the focused workspace can be
updated.

The various komorebi reconciliators and manager modules don't emit
events to subscribers themselves (yet?), so for now we can pass on the
uncloak event.

Maybe we can look into expanding the Notification enum in the future.

fix LGUG2Z#1211
This commit adds a new komorebic command "stackbar-mode" to allow users
to change stackbar modes programmatically.
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.

5 participants