Skip to content

Commit 79dddc0

Browse files
committed
Remove sxProp stories from SelectPanel
1 parent 5dcd46f commit 79dddc0

File tree

3 files changed

+1
-114
lines changed

3 files changed

+1
-114
lines changed

e2e/components/SelectPanel.test.ts

-8
Original file line numberDiff line numberDiff line change
@@ -37,14 +37,6 @@ const scenarios = matrix({
3737
id: 'components-selectpanel-dev--with-css',
3838
name: 'With Css',
3939
},
40-
{
41-
id: 'components-selectpanel-dev--with-sx',
42-
name: 'With Sx',
43-
},
44-
{
45-
id: 'components-selectpanel-dev--with-sx-and-css',
46-
name: 'With Sx and Css',
47-
},
4840
],
4941
})
5042

packages/react/src/SelectPanel/SelectPanel.dev.stories.tsx

-105
Original file line numberDiff line numberDiff line change
@@ -114,108 +114,3 @@ export const WithCss = () => {
114114
</FeatureFlags>
115115
)
116116
}
117-
118-
export const WithSx = () => {
119-
const [selected, setSelected] = useState<ItemInput[]>(items.slice(1, 3))
120-
const [filter, setFilter] = useState('')
121-
const filteredItems = items.filter(
122-
item =>
123-
// design guidelines say to always show selected items in the list
124-
selected.some(selectedItem => selectedItem.text === item.text) ||
125-
// then filter the rest
126-
item.text?.toLowerCase().startsWith(filter.toLowerCase()),
127-
)
128-
// design guidelines say to sort selected items first
129-
const selectedItemsSortedFirst = filteredItems.sort((a, b) => {
130-
const aIsSelected = selected.some(selectedItem => selectedItem.text === a.text)
131-
const bIsSelected = selected.some(selectedItem => selectedItem.text === b.text)
132-
if (aIsSelected && !bIsSelected) return -1
133-
if (!aIsSelected && bIsSelected) return 1
134-
return 0
135-
})
136-
const [open, setOpen] = useState(false)
137-
138-
return (
139-
<FeatureFlags
140-
flags={{
141-
primer_react_css_modules_team: true,
142-
primer_react_css_modules_staff: true,
143-
primer_react_css_modules_ga: true,
144-
}}
145-
>
146-
<FormControl>
147-
<FormControl.Label>Labels</FormControl.Label>
148-
<SelectPanel
149-
title="Select labels"
150-
placeholder="Select labels" // button text when no items are selected
151-
subtitle="Use labels to organize issues and pull requests"
152-
renderAnchor={({children, ...anchorProps}) => (
153-
<Button trailingAction={TriangleDownIcon} {...anchorProps} aria-haspopup="dialog">
154-
{children}
155-
</Button>
156-
)}
157-
open={open}
158-
onOpenChange={setOpen}
159-
items={selectedItemsSortedFirst}
160-
selected={selected}
161-
onSelectedChange={setSelected}
162-
onFilterChange={setFilter}
163-
sx={{fontFamily: 'Times New Roman'}}
164-
/>
165-
</FormControl>
166-
</FeatureFlags>
167-
)
168-
}
169-
170-
export const WithSxAndCSS = () => {
171-
const [selected, setSelected] = useState<ItemInput[]>(items.slice(1, 3))
172-
const [filter, setFilter] = useState('')
173-
const filteredItems = items.filter(
174-
item =>
175-
// design guidelines say to always show selected items in the list
176-
selected.some(selectedItem => selectedItem.text === item.text) ||
177-
// then filter the rest
178-
item.text?.toLowerCase().startsWith(filter.toLowerCase()),
179-
)
180-
// design guidelines say to sort selected items first
181-
const selectedItemsSortedFirst = filteredItems.sort((a, b) => {
182-
const aIsSelected = selected.some(selectedItem => selectedItem.text === a.text)
183-
const bIsSelected = selected.some(selectedItem => selectedItem.text === b.text)
184-
if (aIsSelected && !bIsSelected) return -1
185-
if (!aIsSelected && bIsSelected) return 1
186-
return 0
187-
})
188-
const [open, setOpen] = useState(false)
189-
190-
return (
191-
<FeatureFlags
192-
flags={{
193-
primer_react_css_modules_team: true,
194-
primer_react_css_modules_staff: true,
195-
primer_react_css_modules_ga: true,
196-
}}
197-
>
198-
<FormControl>
199-
<FormControl.Label>Labels</FormControl.Label>
200-
<SelectPanel
201-
title="Select labels"
202-
placeholder="Select labels" // button text when no items are selected
203-
subtitle="Use labels to organize issues and pull requests"
204-
renderAnchor={({children, ...anchorProps}) => (
205-
<Button trailingAction={TriangleDownIcon} {...anchorProps} aria-haspopup="dialog">
206-
{children}
207-
</Button>
208-
)}
209-
open={open}
210-
onOpenChange={setOpen}
211-
items={selectedItemsSortedFirst}
212-
selected={selected}
213-
onSelectedChange={setSelected}
214-
onFilterChange={setFilter}
215-
sx={{fontFamily: 'Times New Roman'}}
216-
className="testCustomClassnameMono"
217-
/>
218-
</FormControl>
219-
</FeatureFlags>
220-
)
221-
}

packages/react/src/SelectPanel/SelectPanel.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ interface SelectPanelBaseProps {
4949
}
5050

5151
export type SelectPanelProps = SelectPanelBaseProps &
52-
Omit<FilteredActionListProps, 'selectionVariant'> &
52+
Omit<FilteredActionListProps, 'selectionVariant' | 'sx'> &
5353
Pick<AnchoredOverlayProps, 'open'> &
5454
AnchoredOverlayWrapperAnchorProps &
5555
(SelectPanelSingleSelection | SelectPanelMultiSelection)

0 commit comments

Comments
 (0)