-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mobile: channel search #3397
mobile: channel search #3397
Conversation
…re query results cause breaking errors
…selected post and posts around on search result click
…tract search results into ui package, fixtures, pull in date formatting util
…a looks right when you get to the bottom
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some initial comments, looking good.
…th the author for display
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good overall! A few nitpicks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! looks good, pending the contact bits we discussed.
search.mp4
Should cover everything from the Figma spec except in message search query highlighting and stitching local/remote results together (remote only right now). The former should just be done and I'll file an issue. For the latter, we'll probably want to hold off until we have a better idea of how the local cache will work. Getting a good UX there with consistent ordering might be tricky.
Still getting sporadic errors/warnings I'm trying to track down and want to include a few tests. Added basic fixtures for a generic input, but didn't have time to take a real stab at building out a reusable Ochre lego.
Fixes LAND-1736