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

Indexfilters search proto just move search #12614

Closed
wants to merge 18 commits into from

Conversation

chloerice
Copy link
Member

WHY are these changes introduced?

Fixes #0000

WHAT is this pull request doing?

How to 🎩

🖥 Local development instructions
🗒 General tophatting guidelines
📄 Changelog guidelines

🎩 checklist

[IndexFilters] Remove search from FilterButton

[IndexFilters] Add a SearchField subcomponent

[IndexFilters] Move search input beside tabs; prototype add as filter UX

[IndexFilters] Update stories to change search placeholder tab reference when active tab changes

[Storybook][Test Pages] Create a new ProductsPage story for prototype

WIP mock saved filter logic (have to fake outside of admin)

switch prototype to tabless filterless orders index

ignore commas when filtering by query filter

Improve order data with David's feedback

Implement fully working tabs and filters to flesh out the table

[Key dev dependencies] Bumped TS and Storybook packages

Ensure special chars like '#' don't break search
@chloerice chloerice force-pushed the indexfilters-search-proto--just-move-search branch from b18535c to 484e1a2 Compare September 6, 2024 17:16
@chloerice chloerice force-pushed the indexfilters-search-proto--just-move-search branch from 484e1a2 to f56cfa2 Compare September 6, 2024 17:20
@chloerice chloerice force-pushed the indexfilters-search-proto--just-move-search branch 2 times, most recently from 06eaf78 to 77e452d Compare September 6, 2024 21:22
@chloerice chloerice closed this Oct 31, 2024
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.

1 participant