Get rid of mouse wheel switch in scene tabs

[KoBeWi]

Editor allows to switch scene tabs with mouse wheel since #47700
However, just like with #65302, 100% of the times I used this functionality it was by accident (and it happens surprisingly often).

This PR adds a interface/scene_tabs/change_tabs_with_mouse_wheel editor setting that disables this function. removes the "feature".

[YuriSizov]

From the RC discussion, I'd just make it activate with a modifier key, like Ctrl, but otherwise leave it on. Would be nice to hear from someone who actually uses this feature though.

[KoBeWi]

Modifier would make this much less discoverable.

[YuriSizov]

Modifier would make this much less discoverable.

I don't think the fact that you can use a mouse wheel over the tabbar is discoverable in the first place, unless by accident. I don't think many expect this to do anything. And if it's not rebindable, then the setting makes it hostile towards the user, i.e. "you either use it like this, or you can't use it at all".

[Calinou]

I wouldn't require using a modifier to use the mouse wheel either, as the behavior then becomes inconsistent with other apps that support this (patched Chromium, some KDE apps like Dolphin, etc). These don't require a modifier to be used.

[YuriSizov]

Should we make the setting disabled by default then? I don't see much demand for this feature, there was no request referenced in the original PR, and the chance to trigger it accidentally is evidently high.

My suggestion for the modifier key is to remove the accidental element without disabling the feature. If we don't want that solution, then disabling it by default is the only option.

[Calinou]

Should we make the setting disabled by default then?

Settings that are disabled by default have poor discoverability – they might as well not exist 🙂

At this point, I would rather remove the feature entirely if you think it's activated accidentally too often. I personally don't rely on it often in Godot (but I do in web browsers).

[KoBeWi]

I changed the PR to remove the feature entirely instead. It technically reverts #47700, but clean revert was not possible due to conflicts, so not sure if it counts 🤔

[fire]

Approving based on the usability improvement and not because of technical review.

[akien-mga]

Thanks!