tree 8145e2a74bc6e0ce992823ef51729256bfb0645e
parent 6e4fb958b5bd7f2d52a080ca3eaef365587b3e3d
author Ben Rohlfs <brohlfs@google.com> 1636039952 +0100
committer Ben Rohlfs <brohlfs@google.com> 1636041766 +0100

Fix regression with the selection-action-box flickering

Change 314023 introduced a regression. The positioning of the action box
would happen before the tooltip is rendered and thus being initially
placed in the wrong position, which unfortunately would make the mouse
pointer hover over the tooltip, thus leading to an invalid selection
range, thus to the tooltip and action box being removed - and then
re-created, seen as a flicker by the user.

The actual fix here is to await the rendering of the tooltip before
positioning the action box.

On top of that we prevent the tooltip from interfering with the
selection.

And we also fix a small bug with `positionBelow` potentially ending up
in a wrong permanent state.

Google-Bug-Id: b/166482375
Change-Id: I9292b7f942fe8f026725988757780894f01bdb11