griefith/test2
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix #135970: Allow Allow Movement After Tooltip Timer Starts

Browse Source 
We can only allow some mouse and pen movement once the tooltip timer
has started. Doing so beforehand can keep some tooltips from starting.

Pull Request: https://projects.blender.org/blender/blender/pulls/135997
This commit is contained in:
Harley Acheson
2025-03-14 19:16:43 +01:00
committed by Harley Acheson
parent a687a02578
commit 18d82c5164
1 changed files with 10 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
source/blender/editors/interface/interface_handlers.cc
View File

@@ -9600,17 +9600,18 @@ static int ui_handle_button_event(bContext *C, const wmEvent *event, uiBut *but)
button_activate_state(C, but, BUTTON_STATE_EXIT);
}
else {
/* Use drag threshold for canceling tooltip on move, even though this is hover.
* By default this allows 3 pixels of movement with a mouse, 10 for tablet pens. */
/* Re-enable tool-tip on mouse move. */
bool reenable_tooltip = true;
bScreen *screen = CTX_wm_screen(C);
if (screen->tool_tip) {
if (screen && screen->tool_tip) {
/* Allow some movement once the tooltip timer has started. */
const int threshold = WM_event_drag_threshold(event);
const int movement = len_manhattan_v2v2_int(screen->tool_tip->event_xy, event->xy);
if (movement > threshold) {
/* Re-enable tool-tip on mouse move. */
ui_blocks_set_tooltips(region, true);
button_tooltip_timer_reset(C, but);
}
const int movement = len_manhattan_v2v2_int(event->xy, screen->tool_tip->event_xy);
reenable_tooltip = (movement > threshold);
}
if (reenable_tooltip) {
ui_blocks_set_tooltips(region, true);
button_tooltip_timer_reset(C, but);
}
}