AModule: Cursor config option

[khaneliman]

Seen a lot of discussion activity around this again... resurrecting the PR #3274 that resolves #3239. Follow up to work done in #3108 to make this opt in instead of always forced on users.

One thing that might be nice is to set a global option for this... but I didn't see a way of doing that easily, atm.

[Alexays]

To avoid yet another breaking change, I prefer to keep the cursor change logic but make it deactivatable in the settings. WDYT?

[khaneliman]

To avoid yet another breaking change, I prefer to keep the cursor change logic but make it deactivatable in the settings. WDYT?

Yeah, i actually agree. I was just waiting for some feedback before I changed it from the original change. I can tweak the change when I get home.

[khaneliman]

@Alexays Alright, implemented fallback to default behavior when someone doesn't provide a configuration.

[khaneliman]

@Alexays Sorry, fixed. Also noticed the mouse enter and leave were a breaking change with tool-tips so moved it back there for the default behavior when not configured.

[khaneliman]

@Alexays Whenever you get a chance

[Alexays]

Thanks!

[haug1]

👏

[haug1]

I was surprised when my waybar crashed coming back from vacation. Turns out it was my own fault in the best kind of way. Had a testing configuration related to these changes left over in my config file from this:

  "cursor": 1231

As I wrote in the man page, attempting to use unavailable cursor options WILL crash your app! 😂