-
Notifications
You must be signed in to change notification settings - Fork 25
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
Filters - Filter Menu - menu header #2951
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@@ -16,5 +17,6 @@ export default meta; | |||
type Story = StoryFn<typeof Filters>; | |||
|
|||
export const BasicExample: Story = () => { | |||
return <Filters label={'test'} />; | |||
// return <Filters label={'test'} />; |
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.
only for testing purposes, I will return back to its state before merging
{props.onSort && ( | ||
<IconButton | ||
size="20" | ||
theme={isActive ? 'info' : 'default'} |
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.
You might notice a slight change between the IconButton when Active. This is because "round" prop is deprecated and I have checked in with Filip to confirm we are using the new prop design(square).
.changeset/dirty-coins-begin.md
Outdated
'@commercetools-uikit/filters': minor | ||
--- | ||
|
||
Implement FilterMenu header component. |
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.
I don't think we need a changeset here, since we aren't publishing filters
for public consumption until the entire feature is done
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.
updated here: 270993d
Thank you
operatorOptions?: Array<TOption>; | ||
onSelectOperand: (value: string) => void; |
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.
This is my fault, because I should have updated the jira ticket, but these should be a single renderOperatorsInput: () => ReactNode
Description
Add Filters - Filter Menu - menu header component.
This pull request introduces a new
FilterMenu
header component and updates related files to support its implementation. The changes include adding new props to theFilterMenu
component, updating tests, and modifying the storybook example.Implementation of
FilterMenu
header component:.changeset/dirty-coins-begin.md
: Added a minor change entry for the newFilterMenu
header component.packages/components/filters/src/filter-menu/header/header.tsx
: Implemented theHeader
component with new props and styles, includinglabel
,operandOptions
,renderOperandsInput
,onSelectOperand
,onSort
, andmenuHeaderWidth
.Updates to
FilterMenu
component:packages/components/filters/src/filter-menu/filter-menu.tsx
: Updated theFilterMenu
component to include the newHeader
component with its props.Test updates:
packages/components/filters/src/filter-menu/header/header.spec.tsx
: Enhanced tests for theHeader
component to cover new functionalities like conditional rendering ofSelectInput
andIconButton
.Storybook update:
This task ideally should have no story so after testing, the story modification will be taken off.