FedeDP / Clightd

A linux bus interface that lets you change screen brightness, compute captured webcam frames brightness and change screen temperature.
GNU General Public License v3.0
82 stars 10 forks source link

[BUG] screen functionality broke fcitx5 candidate window #110

Closed name-snrl closed 7 months ago

name-snrl commented 7 months ago

NOTE: Please try to upgrade Clight if not running on latest release, before reporting a bug!

Clight version the issue has been seen with:

Current version: 4.10-

Used distribution:

NixOS

Describe the bug https://github.com/fcitx/fcitx5/issues/897, but I'm not sure if this can be fixed here, maybe the problem is in the wlroots implementation

Expected behavior candidate window doesn't disappear

To Reproduce https://github.com/fcitx/fcitx5/issues/897

Please attach a Clight verbose log too!

clight.log

FedeDP commented 7 months ago

Hi! Thanks for opening this issue! I think the issue might actually belong to https://github.com/FedeDP/Clightd. Which version of Clightd are you running? I will move this to the Clightd repo btw!

FedeDP commented 7 months ago

I am going to check my screen impl agains eg: https://git.sr.ht/~emersion/grim/tree/master/item/main.c. Perhaps i missed something in the cleanup phase?

Anyway, you are saying that if you disable screen feature in Clight, everything works fine, right?

name-snrl commented 7 months ago

Anyway, you are saying that if you disable screen feature in Clight, everything works fine, right?

Yes, that is correct. I'm also going to open an issue in sway tonight.

name-snrl commented 7 months ago

Which version of Clightd are you running?

Current version: 5.8-

FedeDP commented 7 months ago

I can't see anything wrong in my impl compared to grim; can you try to use grim to take a screenshot and see if it triggers the issue? Thank you very much btw!

name-snrl commented 7 months ago

if it triggers the issue?

lol, yes, it does

FedeDP commented 7 months ago

:rofl: i assume it is not a bug with the client implementation then; perhaps it is a bug on wlroots side?

name-snrl commented 7 months ago

Yeah, now I'm off to sway or wlroots, need to check hyprland. Thank for this investigation.