• Renzo Olivares's avatar
    Add support for double tap and drag for text selection (#109573) · cd0f15a7
    Renzo Olivares authored
    * Replace PanGestureRecognizer in TextSelection with TapAndDragGestureRecognizer
    
    * add tracking of _DragState to new tap_and_drag recognizer and remove some legacy double tap code from text_selection.dart and add logs"
    
    * add dragTapCount, a tap count that is persistent for an entire drag and is set to null on drag end vs the regular tap count which is reset on a timer
    
    * basic double tap to drag functionality and add a local dragTapCount in text_selection.dart to use with the timer callback
    
    * Add offsetFromOrigin and localOffsetFromOrigin to DragUpdateDetails similar to LongPressMoveUpdateDetails, eliminates the need to hold the state of lastDragStartDetails
    
    * make a generic baselongpressgesturerecognizer
    
    * Revert "make a generic baselongpressgesturerecognizer"
    
    This reverts commit aad8f7433bd01e4cd016d527af832c3b1f15fac5.
    
    * rename tap_and_drag to selection_recognizers
    
    * add mixin for consecutivetap
    
    * tap and long press gesture recognizer
    
    * Revert "Revert "make a generic baselongpressgesturerecognizer""
    
    This reverts commit 181350c36718f644eada3e45c1b7b5939f90a340.
    
    * Revert "Revert "Revert "make a generic baselongpressgesturerecognizer"""
    
    This reverts commit 4d69775967858dfd66dd9429e1713da598908a85.
    
    * Add support for secondary button clicks on drag gesture recognizer and separate drag end and tap up callback
    
    * get test running
    
    * rename tapCount to consecutiveTapCount
    
    * dispose timer properly
    
    * add some comments to tests
    
    * Add comments
    
    * Make ConsecutiveTapMixin private and move logic to increment tap count into mixin
    
    * stop tracking pointer when gesture is rejected and detect drags on touch devices
    
    * onCancel for TapAndDrag
    
    * have the TapAndDragGestureRecognizer handle tap downs and tap ups on touch and mouse devices
    
    * add drag to move cursor for android and iOS, and pointer device kind to DragUpdateDetails
    
    * get tests running
    
    * refactor TapAndDragGestureRecognizer moving some logic into _check methods
    
    * Handle cancel properly on TapAndDragGestureRecognizer, having both onTapCancel and onDragCancel, also fix tests
    
    * Fix test mouse drag selects and cannot drag cursor, save _initialPosition based on dragStartBehavior (either on tapDown or dragStart)
    
    * determine if drag has a sufficient global distance to accept and fix some cancel behavior, making _checkCancel clearer
    
    * give up pointer on drag end
    
    * properly stop tracking pointer, fixes test for right click on Apple and non-apple platforms
    
    * clean up some comments from last commit
    
    * remove drag on touch for now
    
    * fix Can select text by dragging with a mouse due to dragStart only being fired on the first PointerMoveEvent, the previous pan gesture recognizer would fire both dragStart and dragUpdate
    
    * Revert "fix Can select text by dragging with a mouse due to dragStart only being fired on the first PointerMoveEvent, the previous pan gesture recognizer would fire both dragStart and dragUpdate"
    
    This reverts commit 124dc79bc3389672c76d7c014ce04edab297abc6.
    
    * correctly use _initialPosition for checkStart and call _checkUpdate after _checkStart if localDelta is not zero
    
    * updates
    
    * fix double tap chains
    
    * Add docs
    
    * Address analyzer
    
    * more analyzer, only issues left are with print statements
    
    * add deadlineTimer to fix conflict with ForcePressGestureRecognizer
    
    * Revert "add deadlineTimer to fix conflict with ForcePressGestureRecognizer"
    
    This reverts commit 3b29ddfff4cde4845edd481ecefb789fea2a0781.
    
    * remove unecessary changes to tests
    
    * secondaryButton should not drag
    
    * Revert "Revert "add deadlineTimer to fix conflict with ForcePressGestureRecognizer""
    
    This reverts commit 0a008f029f5796acd48c17c1897c0b700d5ef3a7.
    
    * updates
    
    * Revert "updates"
    
    This reverts commit 4803b8443a2b67f0b8d29e9a01f712dfcb0f588c.
    
    * Revert "Revert "Revert "add deadlineTimer to fix conflict with ForcePressGestureRecognizer"""
    
    This reverts commit 79251a7af88d5dbb1460a960afc77e65dea18bff.
    
    * fix shift + tap + drag tests, this was happening because a double tap + drag was being registered and not a single tap, added a duration to pumpAndSettle to fix this
    
    * remove TapAndLongPressGestureRecognizer
    
    * fix cupertino text field tests related to shift + tap + drag
    
    * deadline timer try 2
    
    * more logs
    
    * Should reset taps when tap cancel is called, and should wait until gesture is accepted to initiate a drag
    
    * should clear _down and _up when gesture is rejected
    
    * remove erroneous log
    
    * fix selectable text double tap chains test
    
    * dont restart timer until tap up
    
    * reset consecutiveTapCount on drag end
    
    * fix selectableText test
    
    * fix material text field tests
    
    * reject TapAndDragGestureRecognizer when it is neither a tap nor a drag
    
    * remove prints
    
    * clean up
    
    * shift aware
    
    * clean up
    
    * fix cupertino test
    
    * fix text field focus tests
    
    * Add 100ms delay to cupertino test, to prevent a double tap
    
    * clean up test comments
    
    * add comment to test
    
    * uncomment test
    
    * remove longpress changes
    
    * Fix drag on mobile
    
    * remove debug
    
    * Fix drag to move cursor on iOS
    
    * left over from drag fix
    
    * add tests for drag on touch devices
    
    * add test for double tap + drag mouse devices
    
    * add tests
    
    * Fix bug where initialPosition was used before it was set
    
    * Address some review comments and fix issue where if double tap was held too long then long press gesture recognizer would take over
    
    * remove _isDoubleTap flag since it is no longer needed due to previous commit
    
    * Add docs for onTapCancel and onDragCancel
    
    * analyzer fixes
    
    * Do not test selection handles on macOS, since macOS does not support touch
    
    * Add assert for dragStartBehavior
    
    * add double tap + drag tests to cupertino
    
    * use kDoubleTapTimeout instead of const Duration(milliseconds: 300) for readability
    
    * analyzer issues
    
    * update docs
    
    * update more docs
    
    * address comments
    
    * more doc updates
    
    * fix docs
    
    * unused import
    
    * fix docs
    
    * Add more tests
    
    * Add more tests and reject a tap up if we have exceeded the tap tolerance
    
    * updates
    
    * Address comments
    
    * fix test naming
    
    * update documentation
    
    * move selection_recognizers to selection_gestures
    
    * fix analyzer
    
    * fix analyzer
    
    * keysPressedOnDown instead of isShiftPressed
    
    * update docs
    
    * update docs
    
    * Add drag update throttle to TapAndDragGestureRecognizer
    
    * update comments
    
    * missed from merge
    
    * Replace _ConsecutiveTapMixin with _TapStatusTrackerMixin
    
    * updates
    
    * correctly cancel tap when when past tap tolerance with new implementation
    
    * Should call tap and drag cancel if we are giving up a pointer without succesfully tracking a PointerUpEvent
    
    * comments
    
    * move pastTapTolerance to tap tracker
    
    * move pastTapTolerance to tap tracker
    
    * clean up check for nulls and remove use of consecutiveTapCountWhileDragging
    
    * move call to super.acceptGesture to top
    
    * remove print
    
    * clean up
    
    * Fix tests where both PanGestureRecognizer and TapAndDragGestureRecognizer lost
    
    * clean up
    
    * _GestureState -> _DragState
    
    * more docs clean up
    
    * more clean up
    
    * Add onSecondaryTapCancel
    
    * Add docs
    
    * more docs
    
    * Fix broken isPointerAllowed when attempting a right click drag - the _initialButtons is never reset
    
    * revert debug flag
    
    * make primaryPointer private
    
    * Add support for upper count limit in TapAndDragGestureRecognizer, the tap counter should not be allowed to grow infinitely unless that is desired
    
    * fix analyzer
    
    * Use new TapDrag details objects and callbacks
    
    * clean up docs
    
    * clean up and add test for upperLimit
    
    * Add docs for TapAndDragGestureRecognizer and remove some ambiguity of onStart onUpdate and onEnd parameters
    
    * Address review comments
    
    * analyzer fixes
    
    * Call cancel before rejecting the gesture so we can still access _initialButtons
    
    * Recognizer should reject any pointer differing from the original
    
    * Revert "Recognizer should reject any pointer differing from the original"
    
    This reverts commit afd9807480bd11e119bdd2b7d520631511973bab.
    
    * Address reviewer comments
    
    * Correct cancel behavior
    
    * Fix consecutive tap + drag because _dragStart state was not being set when consecutive tap is greater than one
    
    * Add more tests
    
    * Add documentation on behavior with TapGestureRecognizer and DragGestureRecognizer
    
    * more docs
    
    * more docs
    
    * remove comments
    
    * updates
    
    * fix multiple pointer behavior
    
    * only handle the primary pointer
    
    * Clean up dangerous assumptions in gesture details objects
    
    * forgot from rebase
    
    * update docs
    
    * updates
    
    * Clean up some redundant code
    
    * remove whitespace
    
    * fix tests as a result of #115849
    
    * update test docs
    
    * Fix same test from last commit for material variants
    
    * More clean up of redundant code and update docs
    
    * Clean up didStopTrackingLastPointer and untie TapAndDragGestureRecognizer cancel behavior from TapStatusTrackerMixin.currentUp state
    
    * untie pastTapTolerance
    
    * updates
    
    * Add slopTolerance
    
    * update docs
    
    * Have secondary tap handled by TapGestureRecognizer
    
    * update docs
    
    * fix analyzer and address comments
    
    * Add more docs
    
    * Update cancel behavior tol not call on tap cancel when a drag has been accepted
    
    * Change cancel behavior to only cancel if the tap down callback has been sent and merge tapcancel and dragcancel
    
    * update docs;
    
    * Rename selection_gestures to tap_and_drag_gestures
    
    * Address some reviewer comments
    
    * make deadline and slopTolerance private
    
    * updates
    
    * updates
    
    * Address review comments
    
    * remove _initialButtons
    
    * fix docs
    
    * trackTrap -> trackTap
    
    * fix analyzer
    
    * Add test to verify that tap up is called when recognizer accepts before handleEvent is called
    
    * implement Diagnosticable for Details objects;
    
    * sentTapDown == wonArenaForPrimaryPointer, so the implementation now only uses sentTapDown
    
    * Count user tap up immediately and do not wait to win the arena
    
    * Do not need to call super from TapAndDragGestureRecognizer.acceptGesture anymore because mixin implementation is gone
    
    * Do not start selection drag on Android, iOS, and Fuchshsia touch devices if renderEditable does not have focus, this fixes many scubas
    
    * Address reviewer comments
    
    * fix test
    
    * TapAndDragGestureRecognizer should wait for other recognizer to lose before winning the arena
    
    * Address review comments
    
    * Dont check for drag if the start was already found
    
    * Only check for a drag if it has not already been found"
    
    * fix from rebase
    Co-authored-by: 's avatarRenzo Olivares <roliv@google.com>
    cd0f15a7
Name
Last commit
Last update
..
about_test.dart Loading commit data...
action_chip_test.dart Loading commit data...
adaptive_text_selection_toolbar_test.dart Loading commit data...
animated_icons_test.dart Loading commit data...
app_bar_test.dart Loading commit data...
app_bar_theme_test.dart Loading commit data...
app_builder_test.dart Loading commit data...
app_test.dart Loading commit data...
arc_test.dart Loading commit data...
autocomplete_test.dart Loading commit data...
back_button_test.dart Loading commit data...
badge_test.dart Loading commit data...
badge_theme_test.dart Loading commit data...
banner_test.dart Loading commit data...
banner_theme_test.dart Loading commit data...
bottom_app_bar_test.dart Loading commit data...
bottom_app_bar_theme_test.dart Loading commit data...
bottom_navigation_bar_test.dart Loading commit data...
bottom_navigation_bar_theme_test.dart Loading commit data...
bottom_sheet_test.dart Loading commit data...
bottom_sheet_theme_test.dart Loading commit data...
button_bar_test.dart Loading commit data...
button_bar_theme_test.dart Loading commit data...
button_style_test.dart Loading commit data...
button_theme_test.dart Loading commit data...
calendar_date_picker_test.dart Loading commit data...
card_test.dart Loading commit data...
card_theme_test.dart Loading commit data...
checkbox_list_tile_test.dart Loading commit data...
checkbox_test.dart Loading commit data...
checkbox_theme_test.dart Loading commit data...
chip_test.dart Loading commit data...
chip_theme_test.dart Loading commit data...
choice_chip_test.dart Loading commit data...
circle_avatar_test.dart Loading commit data...
color_scheme_test.dart Loading commit data...
colors_test.dart Loading commit data...
data_table_test.dart Loading commit data...
data_table_test_utils.dart Loading commit data...
data_table_theme_test.dart Loading commit data...
date_picker_test.dart Loading commit data...
date_range_picker_test.dart Loading commit data...
debug_test.dart Loading commit data...
desktop_text_selection_toolbar_button_test.dart Loading commit data...
desktop_text_selection_toolbar_test.dart Loading commit data...
dialog_test.dart Loading commit data...
dialog_theme_test.dart Loading commit data...
divider_test.dart Loading commit data...
divider_theme_test.dart Loading commit data...
drawer_test.dart Loading commit data...
drawer_theme_test.dart Loading commit data...
dropdown_form_field_test.dart Loading commit data...
dropdown_menu_test.dart Loading commit data...
dropdown_menu_theme_test.dart Loading commit data...
dropdown_test.dart Loading commit data...
elevated_button_test.dart Loading commit data...
elevated_button_theme_test.dart Loading commit data...
elevation_overlay_test.dart Loading commit data...
expand_icon_test.dart Loading commit data...
expansion_panel_test.dart Loading commit data...
expansion_tile_test.dart Loading commit data...
expansion_tile_theme_test.dart Loading commit data...
feedback_test.dart Loading commit data...
feedback_tester.dart Loading commit data...
filled_button_test.dart Loading commit data...
filled_button_theme_test.dart Loading commit data...
filter_chip_test.dart Loading commit data...
flexible_space_bar_collapse_mode_test.dart Loading commit data...
flexible_space_bar_stretch_mode_test.dart Loading commit data...
flexible_space_bar_test.dart Loading commit data...
floating_action_button_location_test.dart Loading commit data...
floating_action_button_test.dart Loading commit data...
floating_action_button_theme_test.dart Loading commit data...
flutter_logo_test.dart Loading commit data...
grid_title_test.dart Loading commit data...
icon_button_test.dart Loading commit data...
icon_button_theme_test.dart Loading commit data...
icons_test.dart Loading commit data...
inherited_theme_test.dart Loading commit data...
ink_paint_test.dart Loading commit data...
ink_sparkle_test.dart Loading commit data...
ink_splash_test.dart Loading commit data...
ink_well_test.dart Loading commit data...
input_chip_test.dart Loading commit data...
input_date_picker_form_field_test.dart Loading commit data...
input_decorator_test.dart Loading commit data...
list_tile_test.dart Loading commit data...
list_tile_theme_test.dart Loading commit data...
localizations_test.dart Loading commit data...
magnifier_test.dart Loading commit data...
material_button_test.dart Loading commit data...
material_state_mixin_test.dart Loading commit data...
material_state_property_test.dart Loading commit data...
material_states_controller_test.dart Loading commit data...
material_test.dart Loading commit data...
menu_anchor_test.dart Loading commit data...
menu_bar_theme_test.dart Loading commit data...
menu_style_test.dart Loading commit data...
menu_theme_test.dart Loading commit data...
mergeable_material_test.dart Loading commit data...
navigation_bar_test.dart Loading commit data...
navigation_bar_theme_test.dart Loading commit data...
navigation_drawer_test.dart Loading commit data...
navigation_rail_test.dart Loading commit data...
navigation_rail_theme_test.dart Loading commit data...
outlined_button_test.dart Loading commit data...
outlined_button_theme_test.dart Loading commit data...
page_selector_test.dart Loading commit data...
page_test.dart Loading commit data...
page_transitions_theme_test.dart Loading commit data...
paginated_data_table_test.dart Loading commit data...
persistent_bottom_sheet_test.dart Loading commit data...
popup_menu_test.dart Loading commit data...
popup_menu_theme_test.dart Loading commit data...
progress_indicator_test.dart Loading commit data...
radio_list_tile_test.dart Loading commit data...
radio_test.dart Loading commit data...
radio_theme_test.dart Loading commit data...
range_slider_test.dart Loading commit data...
raw_material_button_test.dart Loading commit data...
refresh_indicator_test.dart Loading commit data...
reorderable_list_test.dart Loading commit data...
scaffold_test.dart Loading commit data...
scrollbar_paint_test.dart Loading commit data...
scrollbar_test.dart Loading commit data...
scrollbar_theme_test.dart Loading commit data...
search_test.dart Loading commit data...
segmented_button_test.dart Loading commit data...
segmented_button_theme_test.dart Loading commit data...
selection_area_test.dart Loading commit data...
slider_test.dart Loading commit data...
slider_theme_test.dart Loading commit data...
snack_bar_test.dart Loading commit data...
snack_bar_theme_test.dart Loading commit data...
spell_check_suggestions_toolbar_layout_delegate_test.dart Loading commit data...
spell_check_suggestions_toolbar_test.dart Loading commit data...
stepper_test.dart Loading commit data...
switch_list_tile_test.dart Loading commit data...
switch_test.dart Loading commit data...
switch_theme_test.dart Loading commit data...
tab_bar_theme_test.dart Loading commit data...
tabbed_scrollview_warp_test.dart Loading commit data...
tabs_test.dart Loading commit data...
text_button_test.dart Loading commit data...
text_button_theme_test.dart Loading commit data...
text_field_focus_test.dart Loading commit data...
text_field_helper_text_test.dart Loading commit data...
text_field_restoration_test.dart Loading commit data...
text_field_splash_test.dart Loading commit data...
text_field_test.dart Loading commit data...
text_form_field_restoration_test.dart Loading commit data...
text_form_field_test.dart Loading commit data...
text_selection_test.dart Loading commit data...
text_selection_theme_test.dart Loading commit data...
text_selection_toolbar_test.dart Loading commit data...
text_selection_toolbar_text_button_test.dart Loading commit data...
text_theme_test.dart Loading commit data...
theme_data_test.dart Loading commit data...
theme_defaults_test.dart Loading commit data...
theme_test.dart Loading commit data...
time_picker_test.dart Loading commit data...
time_picker_theme_test.dart Loading commit data...
time_test.dart Loading commit data...
toggle_buttons_test.dart Loading commit data...
toggle_buttons_theme_test.dart Loading commit data...
tooltip_test.dart Loading commit data...
tooltip_theme_test.dart Loading commit data...
tooltip_visibility_test.dart Loading commit data...
typography_test.dart Loading commit data...
user_accounts_drawer_header_test.dart Loading commit data...
value_indicating_slider_test.dart Loading commit data...
will_pop_test.dart Loading commit data...