SelectionArea on iOS should toggle the context menu when tapping on the previous selection

[Renzo-Olivares]

Screen.Recording.2023-08-18.at.2.34.33.AM.mov

Related: #129583

Pre-launch Checklist

• I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
• I read the [Tree Hygiene] wiki page, which explains my responsibilities.
• I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
• I signed the [CLA].
• I listed at least one issue that this PR fixes in the description above.
• I updated/added relevant documentation (doc comments with ///).
• I added new tests to check the change I am making, or this PR is [test-exempt].
• All existing and new tests are passing.

[chunhtai]

LGTM, just have a question

[chunhtai]

How would this interact with the tap to collapse selection in the previous pr?

[Renzo-Olivares]

It would look something like this, replacing the else branch with the _clearSelection.

        instance.onTapUp = (TapUpDetails details) {
          if (defaultTargetPlatform == TargetPlatform.iOS && _positionIsOnActiveSelection(globalPosition: details.globalPosition)) {
            // On iOS when the tap occurs on the previous selection instead of
            // moving the selection, the context menu will be toggled.
            final bool toolbarIsVisible = _selectionOverlay?.toolbarIsVisible ?? false;
            if (toolbarIsVisible) {
              hideToolbar(false);
            } else {
              _showToolbar(location: details.globalPosition);
            }
          } else {
            hideToolbar();
            _selectPositionAt(offset: details.globalPosition);
          }
        };

This toggle behavior still holds true even with the collapsing selection at single tap pr.

[justinmc]

LGTM 👍

[justinmc]

Super readability nit:

Suggested change

	// On iOS when the tap occurs on the previous selection instead of
	// On iOS when the tap occurs on the previous selection, instead of

[auto-submit]

auto label is removed for flutter/flutter/132851, due to - The status or check suite Linux web_canvaskit_tests_0 has failed. Please fix the issues identified (or deflake) before re-applying this label.