ErikReider / SwayNotificationCenter

A simple GTK based notification daemon for SwayWM
GNU General Public License v3.0
1.17k stars 60 forks source link

Broken control center behavior on wayfire #136

Closed ghost closed 2 years ago

ghost commented 2 years ago

https://user-images.githubusercontent.com/80580574/175807336-f4550a28-bb6d-441a-8078-bfd59e44909c.mp4

After the recent updates about BlankWindow control center controls are no longer clickable and intractable. Cannot reproduce on sway.

ErikReider commented 2 years ago

The provided video seems to be corrupted

ErikReider commented 2 years ago

But my initial guess would be that Wayfire handles their layers differently than sway but I'll need a look at the video to be sure

ghost commented 2 years ago

https://user-images.githubusercontent.com/80580574/175807688-ec03420c-8e22-4e9d-8db9-b4b6af41e7d6.mp4

i can play the video here. recorded again just in case ....

ErikReider commented 2 years ago

This is what I see on Firefox: image The Github app displays a broken video and chromium freezes while displaying the initial thumbnail??? Strange

ghost commented 2 years ago

https://user-images.githubusercontent.com/80580574/175807948-deb20dbb-fb3d-4187-b6a1-b02f2e5bc54b.mp4

strange indeed i recorded with wf-recorder earlier. This one is recorded with OBS

ErikReider commented 2 years ago

The video works if I download it and run it through Clapper. The latest works though. Strange...

ghost commented 2 years ago

i have no idea, sorry for the issue

ErikReider commented 2 years ago

Nw! So my uneducated guess would be that Wayfire adds the newest layered windows above the older ones while Sway does the opposite. You can test this by running sleep 5; notify-send "test" "test" and locking the screen. Sway won't show the notification while I'm guessing that Wayfire will?

ghost commented 2 years ago

Yes, notifications appear over swaylock on wayfire

ErikReider commented 2 years ago

What I can do is to expose the click to close as a option in the settings but that is only a half baked solution to this issue

ghost commented 2 years ago

or maybe allow the user to choose layer ? or is it possible for the BlankWindow and ControlCenter Window to be in same window ?

ErikReider commented 2 years ago

Hmmm. The issue with that would be if the user wants to blur the control center but not the translucent clickable background. I'll have to think about this one 🤔

ghost commented 2 years ago

https://github.com/ErikReider/SwayNotificationCenter/blob/01a35a05c2f416fc2796476e2461233d7b59597a/src/controlCenter/controlCenter.vala#L149 replacing GtkLayerShell.Layer.TOP with GtkLayerShell.Layer.OVERLAY i can interact with the control-center and click to close works as well. sway works fine with it as well. One more thing i have noticed now that the BlankWindow also covers waybar even if i have set it to top(not in sway) which i think can be fixed with css styling 20220626_20h54m33s_grim . So, i think allowing the user to choose layer would solve the issue.

ghost commented 2 years ago

After using OVERLAY Layer for control center

Wayfire:

https://user-images.githubusercontent.com/80580574/175821780-751e8e98-61f4-4f6b-b4d9-83aa5b109029.mp4

Sway:

https://user-images.githubusercontent.com/80580574/175821792-6179929c-287d-4ee8-989c-e474dffab844.mp4

ErikReider commented 2 years ago

The only reason why the control center isn't using that layer by default is due to it appearing above the lockscreen on sway. Seems like that won't be the case on Wayfire and hyprland. The layer shell protocol is unstable atm so adding it as a config option isn't ideal if they change the spec in the future.

What would be ideal would be if the lockscreen had its own layer above everything else...

ErikReider commented 2 years ago

I'm cooking up a solution for this. Should be a PR that you can test soon :)

ErikReider commented 2 years ago

Does #139 fix the issue?

ghost commented 2 years ago

Yes, everything is working fine now. Thank You

ArchieSW commented 1 year ago

Same issue on hyprland now

ErikReider commented 1 year ago

Same issue on hyprland now

Are you using the latest changes? (Not the release version)

ArchieSW commented 1 year ago

Same issue on hyprland now

Are you using the latest changes? (Not the release version)

I am using the version from AUR 0.6.3-1

ErikReider commented 1 year ago

I am using the version from AUR 0.6.3-1

Use the -git version

ArchieSW commented 1 year ago

Sorry for late answer. I checked with git version in aur, that contains master version of swaync, and all works correctly.