Page MenuHomePhorge

Stop click leaking from sticker picker
ClosedPublic

Authored by tusooa on May 19 2024, 11:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 18, 3:18 PM
Unknown Object (File)
Wed, Sep 18, 3:18 PM
Unknown Object (File)
Wed, Sep 18, 3:17 PM
Unknown Object (File)
Wed, Sep 18, 3:06 PM
Unknown Object (File)
Sat, Sep 14, 5:46 PM
Unknown Object (File)
Wed, Sep 11, 11:58 AM
Unknown Object (File)
Wed, Sep 11, 2:55 AM
Unknown Object (File)
Sat, Sep 7, 10: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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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