Page MenuHomePhabricator

[web] Keyboard support for typeahead [5/13] - Moved finding suggested users out of tooltip component
ClosedPublic

Authored by przemek on Dec 27 2022, 4:45 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 28, 8:29 AM
Unknown Object (File)
Sat, Oct 26, 8:54 AM
Unknown Object (File)
Mon, Oct 14, 10:48 PM
Unknown Object (File)
Fri, Oct 4, 2:23 PM
Unknown Object (File)
Sep 28 2024, 2:02 PM
Unknown Object (File)
Sep 28 2024, 2:02 PM
Unknown Object (File)
Sep 28 2024, 2:02 PM
Unknown Object (File)
Sep 28 2024, 2:01 PM

Details

Summary

I moved logic user search out of tooltip component. It allows to simplify component and its rendering logic.
Now we only show overlay if we find any users in thread matching typed text.

Test Plan

Typeahead without keyboard support works.
Final tests performed in last diffs.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

web/chat/chat-input-bar.react.js
562–571 ↗(On Diff #20156)

Early exit always please. Please keep this in mind going forward!

inka added a reviewer: tomek.
This revision is now accepted and ready to land.Dec 30 2022, 8:33 AM