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

Mobile search bar #92

Merged
merged 9 commits into from
Feb 6, 2023
Merged

Mobile search bar #92

merged 9 commits into from
Feb 6, 2023

Conversation

lubej
Copy link
Collaborator

@lubej lubej commented Jan 24, 2023

Solution

Add additional variants of search bar:

  • mobile version with icon, instead of search button;
  • expandable search on mobile;

Links

@lubej lubej force-pushed the ml/mobile-search-bar branch 2 times, most recently from bd31e3b to e02736a Compare January 30, 2023 07:45
@lubej lubej requested a review from lukaw3d January 30, 2023 08:15
@lubej lubej marked this pull request as ready for review January 30, 2023 10:44
src/app/components/Search/index.tsx Outdated Show resolved Hide resolved
src/app/components/Search/index.tsx Outdated Show resolved Hide resolved
src/app/components/ExpandableSearch/index.tsx Outdated Show resolved Hide resolved
src/app/components/ExpandableSearch/index.tsx Outdated Show resolved Hide resolved
package.json Show resolved Hide resolved
@lubej lubej force-pushed the ml/mobile-search-bar branch 2 times, most recently from 161aa3d to 73ff81b Compare February 6, 2023 09:21
@lubej lubej requested a review from lukaw3d February 6, 2023 09:53
@lukaw3d
Copy link
Member

lukaw3d commented Feb 6, 2023

Squashed some commits, and merged

@lukaw3d lukaw3d merged commit 3aa8510 into master Feb 6, 2023
@lukaw3d lukaw3d deleted the ml/mobile-search-bar branch February 6, 2023 15:19
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.

None yet

2 participants