Page MenuHomePhabricator

[native] Fix reply not working in search results
ClosedPublic

Authored by inka on May 23 2023, 6:23 AM.
Tags
None
Referenced Files
F3120418: D7942.diff
Fri, Nov 1, 5:33 AM
Unknown Object (File)
Mon, Oct 28, 1:27 AM
Unknown Object (File)
Mon, Oct 28, 1:27 AM
Unknown Object (File)
Mon, Oct 28, 1:27 AM
Unknown Object (File)
Mon, Oct 28, 1:27 AM
Unknown Object (File)
Mon, Oct 28, 1:26 AM
Unknown Object (File)
Mon, Oct 28, 1:16 AM
Unknown Object (File)
Thu, Oct 3, 8:07 PM
Subscribers

Details

Summary

issue: https://linear.app/comm/issue/ENG-3881/reply-doesnt-work-in-search-screen-and-in-pinned-messages-screen
Reply from within the search screen didn't work. We need to navigate back to the thread to be able to reply.
Adding MessageSearchRouteName to threadRoutes is necessary, because we don't want chat input bar to remove its listener for input state events, which happens if activeThreadSelector returns null or an id of a different thread (see isActive variable in chat-input-bar.react.js). activeThreadSelector returns null if threadRoutes doesn't contain the current route name.
ThreadSettingsRouteName is also in threadRoutes, and the search screen is used in a similar fashion to thread settings screen.

Test Plan

Pressed "reply" on the search screen. Checked that the correct thread was opened and the input bar message was prepended with the reply.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka requested review of this revision.May 23 2023, 6:41 AM
inka planned changes to this revision.May 23 2023, 8:15 AM
native/chat/text-message-tooltip-modal.react.js
38 ↗(On Diff #26903)

We already have route variable, we can probably use it.

Thanks for explaining everything!

This revision is now accepted and ready to land.May 23 2023, 11:57 AM