diff options
author | Vova Mshanetskiy <vovams163@gmail.com> | 2019-04-25 18:39:38 +0300 |
---|---|---|
committer | Vova Mshanetskiy <vovams163@gmail.com> | 2019-05-01 12:10:50 +0000 |
commit | a04629d9c2afb0e075a8872793718832200ea26b (patch) | |
tree | 5a45c50a75d87cfaf42d4d4ee198172642e0b0ae /tests | |
parent | 78d0b6e9756c732bf0a7b02d258313a1d2ab461e (diff) |
Android: Fix positioning of selection handles in text editors
The old code used size of m_cursorView to calculate position of the
cursor handle popup. But since on Android layout process is
asynchronous, both width and height of m_cursorView was 0 upon first
call to setPosition(). This resulted in selection handles being
initially displayed at a wrong position for a fraction of second and
then quickly moving to the correct position. In some cases handles
stayed at the wrong position until touched by user.
This patch replaces use of m_cursorView's size with use of m_popup's
size. Width and height of m_popup may be used immediately because they
are explicitly assigned in initOverlay(). The size of m_popup should be
always equal to the would-be size of m_cursorView because of how it is
calculated.
Change-Id: I9868c9a5ce0103d8328b2478cf82feaceba7f404
Reviewed-by: BogDan Vatra <bogdan@kdab.com>
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions