lbonn / rofi

Rofi: A window switcher, run dialog and dmenu replacement - fork with wayland support
Other
943 stars 37 forks source link

[BUG] Rofi-lbonn doesn't respect cursor theme & size in sway #20

Closed Mel34 closed 2 years ago

Mel34 commented 3 years ago

Version

Output of rofi -v Version: 1.6.1-wayland

Launch Command

The commandline used to launch **rofi** rofi -modi drun -show drun

Steps to reproduce

run sway run rofi-lbonn variant in sway

What behaviour you see

What behaviour you expect to see

Additional details:

https://user-images.githubusercontent.com/13254332/105858123-44bb2200-5feb-11eb-837c-4cbc74ca72dc.mp4

digital-mystik commented 3 years ago

I forgot what I ended up doing since it was a while back, but was able to change the cursor to something else (might have been with gsettings, editing settings.ini, setting xcursor in sway's config, or something along those lines), but am unable to downsize the cursor size when hovering over the menu even with everything I can think of set to the desired size. Wonder where rofi-lbonn pulls its cursor settings from, did you ever find a fix?

aacebedo commented 2 years ago

Same issue here, The size and theme of the cursor in lboon rofi is not the same as in other apps. I tried changing the different settings (gsettings, xcursor, settings.ini) without success. That's the only bug I suffer of between the original xorg rofi version and this one.

Sporif commented 2 years ago

The culprit is this line. It should be 24 by default and use the env var XCURSOR_SIZE if set.

For example, this is swaybar's method.

lbonn commented 2 years ago

@Sporif great! Thanks for looking into this!

Would you like to open a PR with the fix?