Skip to content

Commit c6641c8

Browse files
committed
fix(datagrid-date-filter-web): fix e2e test and revert select changes
1 parent d57557a commit c6641c8

File tree

2 files changed

+12
-14
lines changed

2 files changed

+12
-14
lines changed

packages/pluggableWidgets/datagrid-date-filter-web/e2e/DataGridDateFilter.spec.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,8 @@ test.describe("datagrid-date-filter-web", () => {
4545
const filterSelector = page.locator(".filter-selector");
4646
await filterSelector.click();
4747
const filterSelectorOptions = page.locator(".filter-selectors li");
48-
await filterSelectorOptions.first().click();
48+
await filterSelectorOptions.first().click({ delay: 1 });
49+
await expect(page.locator(".filter-selector-button")).toHaveAccessibleName("Between");
4950
const monthSelect = page.locator(".react-datepicker__month-select");
5051
await monthSelect.selectOption("October");
5152
const yearSelect = page.locator(".react-datepicker__year-select");

packages/shared/widget-plugin-filtering/src/controls/filter-selector/FilterSelector.tsx

+10-13
Original file line numberDiff line numberDiff line change
@@ -85,22 +85,19 @@ function useController(props: FilterSelectorProps): ViewProps {
8585

8686
const listboxLabel = props.ariaLabel || "Select filter type";
8787
const buttonLabel = selectedItem?.label || listboxLabel;
88-
// Use memo to prevent max call depth in e2e
89-
// https://github.com/carbon-design-system/carbon/pull/17002
90-
// https://github.com/carbon-design-system/carbon/issues/17001
91-
const buttonProps = useMemo(
92-
() => getToggleButtonProps({ ref: refs.setReference, "aria-label": "none" }),
93-
[getToggleButtonProps, refs.setReference]
94-
);
95-
const listboxProps = useMemo(
96-
() => getMenuProps({ ref: refs.setFloating, "aria-label": "none" }),
97-
[getMenuProps, refs.setFloating]
98-
);
88+
const buttonProps = getToggleButtonProps({
89+
"aria-label": buttonLabel,
90+
ref: refs.setReference
91+
});
92+
const listboxProps = getMenuProps({
93+
"aria-label": listboxLabel,
94+
ref: refs.setFloating
95+
});
9996

10097
return {
10198
open: isOpen,
102-
buttonProps: { ...buttonProps, "aria-label": buttonLabel },
103-
listboxProps: { ...listboxProps, "aria-label": listboxLabel },
99+
buttonProps,
100+
listboxProps,
104101
getItemProps,
105102
selectedItem,
106103
highlightedIndex,

0 commit comments

Comments
 (0)