Page MenuHomePhorge

Stop click leaking from sticker picker
ClosedPublic

Authored by tusooa on May 19 2024, 11:10 AM.
Tags
None
Referenced Files
F112120: D29.1732271780.diff
Thu, Nov 21, 2:36 AM
F112119: D29.1732271778.diff
Thu, Nov 21, 2:36 AM
F112118: D29.1732271772.diff
Thu, Nov 21, 2:36 AM
F112096: D29.1732271261.diff
Thu, Nov 21, 2:27 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