Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
83 commits
Select commit Hold shift + click to select a range
9cee0a4
super wip
broccolinisoup Jul 30, 2024
c7aa5de
just use the actionlist component and revert the type updates
broccolinisoup Jul 30, 2024
40a0c45
Update packages/react/src/FilteredActionList/FilteredActionList.tsx
broccolinisoup Jul 30, 2024
344f6ce
Some more progress
broccolinisoup Jul 31, 2024
fa5f5a6
revert the type changes and cast it :/
broccolinisoup Jul 31, 2024
3cd9ecc
clean
broccolinisoup Jul 31, 2024
0003e59
wip
broccolinisoup Aug 2, 2024
713b5ad
wip wip
broccolinisoup Aug 5, 2024
6fc7f2b
add stories
broccolinisoup Aug 8, 2024
a246981
fix linting
broccolinisoup Aug 8, 2024
e209f47
add tests for groups
broccolinisoup Aug 8, 2024
26cddc1
Merge branch 'main' into select-panel-action-list
broccolinisoup Aug 8, 2024
b1199fb
Merge branch 'select-panel-story-and-tests' into select-panel-action-…
broccolinisoup Aug 8, 2024
b669cff
Map groups
broccolinisoup Aug 8, 2024
1012662
Merge branch 'main' into select-panel-story-and-tests
broccolinisoup Aug 12, 2024
67dc5c9
Update story names for e2e tests
broccolinisoup Aug 12, 2024
c66feb5
oops remove unintended file
broccolinisoup Aug 12, 2024
4fbb3c5
update story name
broccolinisoup Aug 12, 2024
2d5e2b2
same - update story name
broccolinisoup Aug 12, 2024
cee7737
disable animations
broccolinisoup Aug 12, 2024
9c972c3
test(vrt): update snapshots
broccolinisoup Aug 12, 2024
17e4ca8
Merge branch 'select-panel-story-and-tests' into select-panel-action-…
broccolinisoup Aug 12, 2024
26b6fe6
Update tests since new action list has different semantics for group …
broccolinisoup Aug 12, 2024
42c50c0
Merge branch 'main' into select-panel-action-list
broccolinisoup Aug 13, 2024
0d0004f
logging
broccolinisoup Aug 14, 2024
3ba9bc7
pass the rest
broccolinisoup Aug 14, 2024
92ab9c2
extract children and use before text
broccolinisoup Aug 14, 2024
7b3d5e6
remove logging
broccolinisoup Aug 14, 2024
1cf1e09
Merge branch 'main' into select-panel-action-list
siddharthkp Aug 20, 2024
92303d0
test(vrt): update snapshots
siddharthkp Aug 20, 2024
4dc3b41
add active styles to ActionList
siddharthkp Aug 21, 2024
f673034
rename component name to be clearer
siddharthkp Aug 21, 2024
3a7d490
remove variant full from examples
siddharthkp Aug 21, 2024
ab3ea9c
tiny clean up
siddharthkp Aug 21, 2024
44ff862
fix showItemDividers
siddharthkp Aug 21, 2024
46c94e6
another tiny cleanup
siddharthkp Aug 21, 2024
ff8379b
pull MappedActionListItem to make it stable
siddharthkp Aug 22, 2024
9353d71
Merge branch 'main' into select-panel-action-list
siddharthkp Aug 22, 2024
0a126ad
test(vrt): update snapshots
siddharthkp Aug 22, 2024
91cd5a3
show active styles only when used with keyboard
siddharthkp Aug 22, 2024
40322c5
backward compat: expose id as data-id
siddharthkp Aug 22, 2024
f0150e4
Merge branch 'select-panel-action-list' of github.com:primer/react in…
siddharthkp Aug 22, 2024
f72b9d4
update snapshots
siddharthkp Aug 22, 2024
85d0237
add story for long strings
siddharthkp Aug 26, 2024
2eca9fa
fishing for errors
siddharthkp Aug 26, 2024
c875223
backward compatibility for renderItem
siddharthkp Aug 26, 2024
02a6dc9
remove todo now
siddharthkp Aug 26, 2024
25c574a
add a feature flag
siddharthkp Aug 26, 2024
5dc6adb
clean up dual filter list setup
siddharthkp Aug 27, 2024
85e0bb4
run jests test with both states of feature flags
siddharthkp Aug 27, 2024
0ee74e1
refactor snapshot tests with scenarios
siddharthkp Aug 27, 2024
798008e
remove feature flag for main
siddharthkp Aug 27, 2024
beca8db
Merge branch 'main' into select-panel-action-list
siddharthkp Aug 27, 2024
14fd206
Merge branch 'selectpanel-prepare-scenarios' into select-panel-action…
siddharthkp Aug 27, 2024
185eb17
test(vrt): update snapshots
siddharthkp Aug 28, 2024
318aa99
add feature flag to e2e matrix
siddharthkp Aug 28, 2024
e9c523c
test(vrt): update snapshots
siddharthkp Aug 28, 2024
efd1216
backward compat: allow groupMetadata to be empty array
siddharthkp Aug 28, 2024
cd9a931
Merge branch 'select-panel-action-list' of github.com:primer/react in…
siddharthkp Aug 28, 2024
619ff96
Merge branch 'main' into select-panel-action-list
siddharthkp Aug 29, 2024
e3fe7b8
sigh newline
siddharthkp Aug 29, 2024
ebac341
Create sour-cooks-dress.md
siddharthkp Aug 30, 2024
0513474
Merge branch 'main' into select-panel-action-list
siddharthkp Aug 30, 2024
2afa236
copy SelectPanel snapshots from main
siddharthkp Sep 2, 2024
0f38bf4
remove unrelated changes in this PR
siddharthkp Sep 3, 2024
f72aa55
test(vrt): update snapshots
siddharthkp Sep 3, 2024
9d6ecdc
add more bindkeys
siddharthkp Sep 3, 2024
e33bc33
add for deprecated version as well
siddharthkp Sep 3, 2024
fef1c27
label listbox/ActionList by panel title
siddharthkp Sep 3, 2024
4b2f9c6
Create wicked-books-occur.md
siddharthkp Sep 3, 2024
d20fafb
remove Home and End
siddharthkp Sep 4, 2024
44cd938
add test for PageDown and PageUp
siddharthkp Sep 4, 2024
04439b5
update changeset
siddharthkp Sep 4, 2024
c4d7cab
active styles for both directly and indirectly
siddharthkp Sep 4, 2024
de4684c
add role=combobox
siddharthkp Sep 5, 2024
540ef9f
update snapshot
siddharthkp Sep 5, 2024
058bd56
Merge branch 'main' into select-panel--functional-adjustments-1
siddharthkp Sep 10, 2024
5f862ed
test(vrt): update snapshots
siddharthkp Sep 10, 2024
aab2b3d
remove features from deprecated version
siddharthkp Sep 11, 2024
3c9c6c5
Merge branch 'select-panel--functional-adjustments-1' of github.com:p…
siddharthkp Sep 11, 2024
5f9fdef
update test
siddharthkp Sep 11, 2024
b7a81e3
only use aria-labelledby when aria-label is not passed
siddharthkp Sep 11, 2024
dafe94e
remove combobox for this PR
siddharthkp Sep 11, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add tests for groups
  • Loading branch information
broccolinisoup committed Aug 8, 2024
commit e209f472f65feea66bdefc7f7354576e0a6eb72d
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ export const WithPlaceholderSelect = () => {
{children ?? 'Select Labels'}
</Button>
)}
placeholder="Please select options below"
placeholder="Select issue labels"
open={open}
onOpenChange={setOpen}
items={filteredItems}
Expand Down
119 changes: 119 additions & 0 deletions packages/react/src/SelectPanel/SelectPanel.test.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import {render, screen} from '@testing-library/react'
import React from 'react'
import {SelectPanel, type SelectPanelProps} from '../SelectPanel'
import type {ItemInput, GroupedListProps} from '../deprecated/ActionList/List'
import {userEvent} from '@testing-library/user-event'
import ThemeProvider from '../ThemeProvider'

Expand Down Expand Up @@ -373,4 +374,122 @@ describe('SelectPanel', () => {
expect(screen.getByText('test footer')).toBeVisible()
})
})

const listOfItems: Array<ItemInput> = [
{
id: '1',
key: 1,
text: 'Item 1',
groupId: '1',
},
{
id: '2',
key: 2,
text: 'Item 2',
groupId: '1',
},
{
id: '3',
key: 3,
text: 'Item 3',
groupId: '2',
},
{
id: '4',
key: 4,
text: 'Item 4',
groupId: '3',
},
]

const groupMetadata: GroupedListProps['groupMetadata'] = [
{groupId: '1', header: {title: 'Group title 1'}},
{groupId: '2', header: {title: 'Group title 2'}},
{groupId: '3', header: {title: 'Group title 3'}},
]

function SelectPanelWithGroups() {
const [selectedItems, setSelectedItems] = React.useState<SelectPanelProps['items']>([])
const [open, setOpen] = React.useState(false)
const [filter, setFilter] = React.useState('')

const onSelectedChange = (selections: ItemInput[]) => {
setSelectedItems(selections)
}

return (
<ThemeProvider>
<SelectPanel
title="test title"
subtitle="test subtitle"
groupMetadata={groupMetadata}
placeholderText="Filter items"
placeholder="Select items"
items={listOfItems}
selected={selectedItems}
onSelectedChange={onSelectedChange}
open={open}
onOpenChange={isOpen => {
setOpen(isOpen)
}}
filterValue={filter}
onFilterChange={value => {
setFilter(value)
}}
/>
</ThemeProvider>
)
}

describe('with groups', () => {
it('should render groups with items', async () => {
const user = userEvent.setup()

render(<SelectPanelWithGroups />)

await user.click(screen.getByText('Select items'))
const listbox = screen.getByRole('listbox')
expect(listbox).toBeVisible()
expect(listbox).toHaveAttribute('aria-multiselectable', 'true')

// listbox should has 3 gorups and each have heading
const headings = screen.getAllByRole('heading')

// The first heading is the h1 and it is the title of the dialog
expect(headings[1]).toHaveTextContent('Group title 1')
expect(headings[2]).toHaveTextContent('Group title 2')
expect(headings[3]).toHaveTextContent('Group title 3')

expect(screen.getAllByRole('option')).toHaveLength(4)
})
it('should select items within groups', async () => {
const user = userEvent.setup()

render(<SelectPanelWithGroups />)

await user.click(screen.getByText('Select items'))

// Select the first item
await user.click(screen.getByRole('option', {name: 'Item 1'}))
expect(
screen.getByRole('option', {
name: 'Item 1',
}),
).toHaveAttribute('aria-selected', 'true')

await user.click(screen.getByRole('option', {name: 'Item 3'}))
expect(
screen.getByRole('option', {
name: 'Item 3',
}),
).toHaveAttribute('aria-selected', 'true')

await user.click(screen.getByRole('option', {name: 'Item 4'}))
expect(
screen.getByRole('option', {
name: 'Item 4',
}),
).toHaveAttribute('aria-selected', 'true')
})
})
})