Page MenuHomePhorge

Stop click leaking from sticker picker
ClosedPublic

Authored by tusooa on May 19 2024, 11:10 AM.
Tags
None
Referenced Files
F60617520: D29.1775707930.diff
Tue, Apr 7, 9:12 PM
F60594296: D29.1775705703.diff
Tue, Apr 7, 8:35 PM
F60491380: D29.1775695797.diff
Tue, Apr 7, 5:49 PM
F60426788: D29.1775689533.diff
Tue, Apr 7, 4:05 PM
F60426752: D29.1775689529.diff
Tue, Apr 7, 4:05 PM
F60199599: D29.1775667518.diff
Tue, Apr 7, 9:58 AM
Subscribers
None

Details

Summary

The current use of TapHandler will leak the click event to the ones beneath them. This can cause, say a username to be clicked on when the user is picking a sticker, leading to an unwanted mention being triggered. MouseArea does not suffer from the same problem as TapHandler, and it is able to handle touch events as well.

Type: fix

Test Plan

Open the sticker picker and click on the area where there is both a sticker and the underlying layer (i.e. the timeline) has a sender username. Verify that the sticker is sent and the user mention is not in the send message box.

Diff Detail

Repository
rK kazv
Branch
tusooa/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 45
Build 79: GitLab CI for kazv
Build 78: arc lint + arc unit

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.May 19 2024, 11:13 AM
Harbormaster failed remote builds in B45: Diff 76!
This revision is now accepted and ready to land.May 29 2024, 5:46 AM