Skip to content
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

ALF text input for generic search input #5511

Merged
merged 9 commits into from
Sep 30, 2024
Merged

Conversation

mozzius
Copy link
Member

@mozzius mozzius commented Sep 26, 2024

Have not tested super thoroughly

Screenshot 2024-09-26 at 21 31 52

@arcalinea arcalinea temporarily deployed to samuel/alf-mini-search - social-app PR #5511 September 26, 2024 20:32 — with Render Destroyed
Copy link

github-actions bot commented Sep 26, 2024

Old size New size Diff
7.9 MB 7.9 MB -580 B (-0.01%)

* origin/main:
  Use Inter variable font (#5540)
  Fix name in `feature_request.yml` (#5542)
  Fix sticky offset, gear color (#5537)
  Move email test to root tests dir (#5527)
  revamp issue templates, add new arch template (#5532)
  use PressableScale for animation (#5541)
  Rework native autocomplete (#5521)
  Pinned posts (#5055)
  Remove Segment (#5518)
  Fix alignment of cancel button on search (#5520)
  [Share Extension] Support on Android for sharing videos to app (#5466)
  Ignore bogus onScroll values (#5499)
  add podcasts to spotify embeds (#5514)
  Tweak font size of "Write your reply" (#5513)
  Lazy load ViewShot (#5517)
  [Share Extension] Support images/movies from other apps like iMessage (#5515)
@arcalinea arcalinea temporarily deployed to samuel/alf-mini-search - social-app PR #5511 September 30, 2024 21:42 — with Render Destroyed
Copy link
Member

@estrattonbailey estrattonbailey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice one! Thanks for doing the legwork here. I just tweaked the new component and some spacing around its usages. Plus swapped into the search page now that the dust has settled there.

@estrattonbailey estrattonbailey merged commit 2129f69 into main Sep 30, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants