Prevent accidental drags by adding drag distance threshold

Co-authored-by: fox <12120644+foxydevloper@users.noreply.github.com>
This commit is contained in:
Hugo Locurcio 2021-07-16 06:03:09 +02:00
parent 08d517243c
commit 44b01751e6
No known key found for this signature in database
GPG key ID: 39E8F8BE30B0A49C
4 changed files with 37 additions and 15 deletions

View file

@ -2491,6 +2491,25 @@ bool CanvasItemEditor::_gui_input_select(const Ref<InputEvent> &p_event) {
// Start dragging
if (still_selected) {
// Drag the node(s) if requested
drag_start_origin = click;
drag_type = DRAG_QUEUED;
}
// Select the item
return true;
}
}
}
if (drag_type == DRAG_QUEUED) {
if (b.is_valid() && !b->is_pressed()) {
drag_type = DRAG_NONE;
return true;
}
if (m.is_valid()) {
Point2 click = transform.affine_inverse().xform(m->get_position());
// Scale movement threshold with zoom (which itself is set relative to the editor scale).
bool movement_threshold_passed = drag_start_origin.distance_to(click) > (8 * MAX(1, EDSCALE)) / zoom;
if (m.is_valid() && movement_threshold_passed) {
List<CanvasItem *> selection2 = _get_edited_canvas_items();
drag_selection.clear();
@ -2505,8 +2524,6 @@ bool CanvasItemEditor::_gui_input_select(const Ref<InputEvent> &p_event) {
drag_from = click;
_save_canvas_item_state(drag_selection);
}
}
// Select the item
return true;
}
}

View file

@ -211,6 +211,7 @@ private:
DRAG_ANCHOR_BOTTOM_RIGHT,
DRAG_ANCHOR_BOTTOM_LEFT,
DRAG_ANCHOR_ALL,
DRAG_QUEUED,
DRAG_MOVE,
DRAG_SCALE_X,
DRAG_SCALE_Y,
@ -390,6 +391,7 @@ private:
Control *top_ruler;
Control *left_ruler;
Point2 drag_start_origin;
DragType drag_type;
Point2 drag_from;
Point2 drag_to;

View file

@ -1206,6 +1206,7 @@ void SpatialEditorViewport::_sinput(const Ref<InputEvent> &p_event) {
}
_edit.mouse_pos = b->get_position();
_edit.original_mouse_pos = b->get_position();
_edit.snap = spatial_editor->is_snap_enabled();
_edit.mode = TRANSFORM_NONE;
@ -1412,7 +1413,8 @@ void SpatialEditorViewport::_sinput(const Ref<InputEvent> &p_event) {
} else if (nav_scheme == NAVIGATION_MODO && m->get_alt()) {
nav_mode = NAVIGATION_ORBIT;
} else {
if (clicked) {
const bool movement_threshold_passed = _edit.original_mouse_pos.distance_to(_edit.mouse_pos) > 8 * EDSCALE;
if (clicked && movement_threshold_passed) {
if (!clicked_includes_current) {
_select_clicked(clicked_wants_append, true);
// Processing was deferred.

View file

@ -357,6 +357,7 @@ private:
Vector3 orig_gizmo_pos;
int edited_gizmo;
Point2 mouse_pos;
Point2 original_mouse_pos;
bool snap;
Ref<EditorSpatialGizmo> gizmo;
int gizmo_handle;