Editor panning scheme should be Scroll Pans by default on laptop

[ssambender]

When on a laptop/trackpad, I find the editor settings very unintuitive. Since the typical pinch/zoom gesture pans side to side instead of zooming, as well as the two-finger drag (pan) gesture zooming in and out. Essentially the controls are reversed when on laptop, but I understand with a normal mouse it makes sense.

In Editor -> Editor Settings -> Editors, there are options to change these, and updating the top two enums from Scroll Zooms to Scroll Pans fixes this issue.

editor_settings.cpp:

EDITOR_SETTING_BASIC(Variant::INT, PROPERTY_HINT_ENUM, "editors/panning/2d_editor_panning_scheme", 0, "Scroll Zooms,Scroll Pans");
EDITOR_SETTING_BASIC(Variant::INT, PROPERTY_HINT_ENUM, "editors/panning/sub_editors_panning_scheme", 0, "Scroll Zooms,Scroll Pans");
EDITOR_SETTING_BASIC(Variant::INT, PROPERTY_HINT_ENUM, "editors/panning/animation_editors_panning_scheme", 1, "Scroll Zooms,Scroll Pans");

Is there a way to go about making the enums set to Scroll Pans by default when the detected input is trackpad/touchpad.

Something with the ImGui::GetIO().MouseWheel I would assume, just not sure as to where that part of the code would be inserted.

[Calinou]

Is there a way to go about making the enums set to Scroll Pans by default when the detected input is trackpad/touchpad.

I don't think Godot can detect whether the mouse input is coming from a touchpad or not.

[ssambender]

Currently I don't think it does, but C++ provides the capability with pretty minimal implementation. See the ImGui::GetIO().MouseWheel func above.

[AThousandShips]

What is ImGui? It doesn't look like any library we are using

[Zireael07]

@AThousandShips ImGui is a pretty famous GUI library.

[AThousandShips]

Okay, but we don't use it, as we have our own gui library, so the platform specific things would still need to be implemented

Edit: glancing at the code of ImGui it doesn't even support getting touchpad separated from mouse on Windows, it doesn't even seem to be natively supported, it doesn't even seem to work that way with the wheel