✨ Enable reverse cycling of window actions with Shift key

[Morniak]

This is an initial implementation for #696, adding support for cycling backwards through actions when the Shift key is pressed.

With this code, backward cycling is enabled for all Cycle actions if the Shift key is pressed and neither the trigger key nor the keybind already includes Shift.

To support this behavior, I introduced an isShiftPressed() method in KeybindMonitor. It's a bit of a rough solution, so if you have a cleaner approach in mind, I'm happy to revise it.

Also, we should discuss the feature's scope. It might make sense to add a global toggle to enable/disable backward cycling or add per-cycle configuration of this behavior

Feedback is welcome—especially on implementation design and configurability.

[MrKai77]

@Morniak actually, this looks great so far! You mentioned isShiftPressed() feeling like a rough solution, but I don’t think it is :)
Since KeybindMonitor was designed to track pressed keys during Loop’s activation, it totally makes sense to use it here.

As for the feature’s scope, I think it should be a global setting. Making it per-cycle might overcomplicate things for the average user.

[Morniak]

Great, thanks! Should I go ahead and add a global toggle to disable this feature?
If so, do you have a preferred location for it? I was thinking the Keybinds tab, under the Settings section, might be a good fit.

[MrKai77]

Yes, that makes sense. But It would also make sense to make it it's own little "section", without a title, right below the trigger settings. And, do you think it would make sense to only show it if the keybinds list contains a cycle action as well?

[Morniak]

A new section below the trigger key makes sense, and showing it only when there's a cycle action is a good idea — I'm on it.
If the trigger key uses Shift, I think we should keep the toggle visible but disabled, and add a LuminareInfoView to explain why.

[Morniak]

The change has been pushed, you can check it out now @MrKai77.
Note: the section is toggled off, not disabled, when the trigger key includes Shift because LuminareInfoView doesn’t appear for disabled LuminareToggle.