Bug 466048 - [Regression] Bismuth tiling manager does not detect windows on Wayland
Summary: [Regression] Bismuth tiling manager does not detect windows on Wayland
Status: RESOLVED DOWNSTREAM
Alias: None
Product: kwin
Classification: Plasma
Component: wayland-generic (other bugs)
Version First Reported In: git master
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: KWin default assignee
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2023-02-19 03:24 UTC by Anil
Modified: 2023-02-22 03:54 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Anil 2023-02-19 03:24:19 UTC 
SUMMARY
As of Plasma 5.27, the Bismuth Tiling addon (https://github.com/Bismuth-Forge/bismuth/issues/473)
does not detect windows properly on wayland.

It only sporadically detects when windows are opened, and it never detects when windows are closed. Therefore these windows don't get tiled.

Note that X11 still works properly

STEPS TO REPRODUCE
1. Tried on both 5.27 and git master
2. Install the KDE bismuth addon
3. Enable the addon in settings
4. Logout/login just to be safe
5. Open and close applications (such as dolphin, system settings etc).
OBSERVED RESULT
Bismuth does not detect the presence of the windows

EXPECTED RESULT
Windows are tiled or untiled as appropriate


SOFTWARE/OS VERSIONS
Windows: 
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
Comment 1 David Edmundson 2023-02-19 22:47:22 UTC 
Having read the upstream ticket:
 - it's not detecting the windows because it's hitting a try-catch fail *before* it binds to the other events of the window
 - hitting an error in a big try/catch block very strongly points to an error in the script itself 

Whilst clearly some behaviour change has happened in kwin, right now this looks to be on the bismuth side until something more specific is narrowed down. Please do update here if something more specific points to kwin.