Skip to content
This repository has been archived by the owner on Jun 16, 2024. It is now read-only.

[Bug]: Focus Bottom Window hotkey focuses wrong window #221

Open
jsjuris89 opened this issue Dec 5, 2021 · 15 comments
Open

[Bug]: Focus Bottom Window hotkey focuses wrong window #221

jsjuris89 opened this issue Dec 5, 2021 · 15 comments
Labels
bug Something isn't working needs more info The issue needs more information from the submitter side

Comments

@jsjuris89
Copy link

Summary

Meta + K or Focus Bottom Window doesn't go to correct window, rest Focus hotkeys work as expected.

Steps to Reproduce

  1. Tile Layout 2
  2. 4 windows open
  3. Try go from top window to bottom window
  4. Goes wrong (see screenshot)

Expected behavior

Should go to bottom window.

Screenshots


Pressing Meta + K on 1st window goes to 4th (should go 2nd) window.
Pressing Meta + K on 3rd window goes to 2nd (should go 4th) window

Bismuth version

AUR - kwin-bismuth 2.2.0-1

KDE Plasma version

5.23.4

The platform KWin is running on

X11

Additional context

No response

@jsjuris89 jsjuris89 added the bug Something isn't working label Dec 5, 2021
@gikari
Copy link
Member

gikari commented Dec 5, 2021

Cannot reproduce, but It is possible, that this behavior could break due to #209 or #208.
If anyone else could reproduce and/or make more precise instructions on how to reproduce (there might be a difference in our configurations, that prevents reproducing), I will take a look.

@gikari gikari added the needs more info The issue needs more information from the submitter side label Dec 5, 2021
@gikari
Copy link
Member

gikari commented Dec 5, 2021

@jsjuris89 Could you reproduce your issue with #222 applied?

@jsjuris89
Copy link
Author

@gikari I could try. What I need to do?

@gikari
Copy link
Member

gikari commented Dec 5, 2021

Since it was just merged, you need to:

  1. Remove AUR version.
  2. Run these (make sure you have NPM installed):
    git clone https://github.com/Bismuth-Forge/bismuth.git
    cd bismuth
    npm install # Installs dependencies for building
    npm run sysdep-install # Install system dependencies for building
    npm run bi-install # Installs all Bismuth components
  3. Try to reproduce the bug.

You may want to roll back to the AUR version after that, to do that, uninstall the Bismuth with:

npm run bi-uninstall

@jsjuris89
Copy link
Author

@gikari I got this error:

[elnath@elnath-ryzen bismuth]$ npm run sysdep-install

> [email protected] sysdep-install
> scripts/sysdep-install.sh

⛓️ Installing system dependencies...
⚠ Your distribution is EndeavourOS, but you have to install system dependencies manually.
[elnath@elnath-ryzen bismuth]$

@gikari
Copy link
Member

gikari commented Dec 5, 2021

@gikari I got this error:

[elnath@elnath-ryzen bismuth]$ npm run sysdep-install

> [email protected] sysdep-install
> scripts/sysdep-install.sh

⛓️ Installing system dependencies...
⚠ Your distribution is EndeavourOS, but you have to install system dependencies manually.
[elnath@elnath-ryzen bismuth]$

You can open an issue about it. For now, just execute this:

      sudo pacman -S --noconfirm --needed \
        cmake ninja extra-cmake-modules

Then continue with the rest of commands.

@jsjuris89
Copy link
Author

@gikari Ok I did that. Still have same bug.

@gikari
Copy link
Member

gikari commented Dec 5, 2021

I forgot to mention, that you need to restart kwin with:

kwin_x11 --replace & disown

If you didn't do that, the Bismuth script is not up-to-date.
You can of course log in and log out to do the same.

@jsjuris89
Copy link
Author

@gikari I tried both log out and the command above. Still same behaviour. Only Focus bottom window is bugged.
Here is additional info:

If I keep pressing Meta + K it goes from 1 -> 4 -> 3 -> 2 -> 1

@gikari
Copy link
Member

gikari commented Dec 5, 2021

This is really weird. It's very similar to the behavior of the "Previous Window" shortcut. Are you sure Meta K is bound correctly?

You can check it by going to "System Setting" > "Shortcuts" > "KWin" and filtering the results by "Bismuth" keyword.

@jsjuris89
Copy link
Author

jsjuris89 commented Dec 5, 2021

@gikari Alright, we getting closer, this is what I found in hotkeys:

So I reassigned Meta K to Bottom Window.

The problem is that hotkey is not working. Pressing Meta + K does nothing.

Edit1: But setting it to Meta + N for example, the hotkey is working. Just Meta + K is NOT working. Any ideas why that is so?

@gikari
Copy link
Member

gikari commented Dec 5, 2021

KWin shortcuts can be buggy when reassigning from old shortcuts. Close and reopen the same shortcut dialog and check if the shortcut is bound.

Edit (in response to sudden Edit): You could try the rebinding to Meta + K again with the double check from above.

@jsjuris89
Copy link
Author

Yes, I cleared Focus bottom window to no hotkey, hit applied. And tried again Meta + K now it works. So the problem was not in the bismuth but there are some bugs in System Setting > Shurtcuts ? @gikari

@gikari
Copy link
Member

gikari commented Dec 5, 2021

I don't know, lol. The rebinding from one shortcut to another was random and from what I checked, there isn't anything, that could cause that in the Bismuth code base.

Could you please report the non-applying-rebinding bug to upstream kwin and link it here? You need to go to https://bugs.kde.org to do that.

KWin shortcuts can be buggy when reassigning from old shortcuts. Close and reopen the same shortcut dialog and check if the shortcut is bound.

@jsjuris89
Copy link
Author

jsjuris89 commented Dec 5, 2021

Could you please report the non-applying-rebinding bug to upstream kwin and link it here?

Sure I can try, but tomorrow sleep time now.

Yeah its weird. Move Window Down command also has no hotkey.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working needs more info The issue needs more information from the submitter side
Projects
None yet
Development

No branches or pull requests

2 participants