fix: Remove redundant onUpdatePopover calls, #157

5app · Sep 14, 2021 · a7fd14d · a7fd14d
1 parent bc67278
commit a7fd14d
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 5 deletions.
diff --git a/src/Combobox/ComboboxMenu.js b/src/Combobox/ComboboxMenu.js
@@ -19,7 +19,6 @@ function ComboboxMenu({children, 'aria-label': ariaLabel}) {
 			{...popover.props}
 			ref={popover.setRef}
 			arrow={popover.arrow}
-			onUpdatePopover={popover.update}
 		>
 			{shouldShowStatusMessage ? (
 				<Box px="s" py="xs">

diff --git a/src/Menu/MenuList.js b/src/Menu/MenuList.js
@@ -17,7 +17,6 @@ function MenuList({children}) {
 			{...popover.props}
 			ref={popover.setRef}
 			arrow={popover.arrow}
-			onUpdatePopover={popover.update}
 		>
 			<MenuListUI.Wrapper {...menuListProps}>
 				{children}

diff --git a/src/NavMenu/NavMenuList.js b/src/NavMenu/NavMenuList.js
@@ -18,7 +18,6 @@ function NavMenuList({children}) {
 			ref={popover.setRef}
 			id={itemList.listId}
 			arrow={popover.arrow}
-			onUpdatePopover={popover.update}
 		>
 			<MenuListUI.Wrapper>{children}</MenuListUI.Wrapper>
 		</PopoverCard>

diff --git a/src/Popover/index.js b/src/Popover/index.js
@@ -43,7 +43,6 @@ const Popover = forwardRef((props, ref) => {
 				{...popover.props}
 				ref={popover.setRef}
 				arrow={popover.arrow}
-				onUpdatePopover={popover.update}
 			>
 				{content}
 			</PopoverCard>

diff --git a/src/Select/Select.js b/src/Select/Select.js
@@ -143,7 +143,6 @@ function Select({
 				{...popover.props}
 				ref={popover.setRef}
 				arrow={popover.arrow}
-				onUpdatePopover={popover.update}
 			>
 				<MenuListUI.Wrapper
 					role="listbox"