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

Misc stuff #321

Merged
merged 3 commits into from
Mar 3, 2024
Merged

Misc stuff #321

merged 3 commits into from
Mar 3, 2024

Conversation

Leleat
Copy link
Owner

@Leleat Leleat commented Feb 27, 2024

No description provided.

Leleat added 3 commits February 26, 2024 17:22
PS. the split into a default mode as a radio group and the other modes
having a dropdown for choosing the modifier is a bit awkward (and it
made the code awkward as well when the default mode also has a modifer
set). It's probably better that each mode gets a dropdown and 'default'
being one option for in the dropdown... Oh well, future stuff.

Closes #266
While I can't reproduce the issue, multiple people reported the bug (
via multiple issues). Apparently, disabling the tiling animations works
around the bug. So disable the tiling animations for layouts. A proper
fix would probably require implementing custom animations rather than
mis-using/hooking into GNOME Shell animations. For now just use this
simple workaround since this is a rather annoying bug for those who are
affected and has been around for a while.

Works around #304
Currently, a conflict of the window action key (Alt or Super) with a
movement mode modifier of T-A is only checked and handled during enable
time. But users may change the window action key while T-A is already
enabled. Handle that case now as well. Previously, the modifier for the
conflicting move mode was switched to the non-window-action-modifier.
But that can be an issue, if that modifier is already taken by a
different mode. So simply disable the modifier for the conflicting mode.

Closes #313
@Leleat Leleat merged commit 58a80dc into main Mar 3, 2024
3 checks passed
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.

1 participant