Use transitionDuration 0 on both popovers and menus

GMOD · Feb 1, 2023 · e07c518 · e07c518
1 parent 3bddafb
commit e07c518
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 6 deletions.
diff --git a/packages/core/ui/DropDownMenu.tsx b/packages/core/ui/DropDownMenu.tsx
@@ -7,9 +7,6 @@ import ArrowDropDown from '@mui/icons-material/ArrowDropDown'
 import Menu, { MenuItem } from './Menu'
 
 const useStyles = makeStyles()(theme => ({
-  root: {
-    display: 'flex',
-  },
   buttonRoot: {
     '&:hover': {
       backgroundColor: alpha(
@@ -51,7 +48,7 @@ function DropDownMenu({
   }
 
   return (
-    <div className={classes.root}>
+    <>
       <Button
         ref={anchorEl}
         onClick={handleToggle}
@@ -70,7 +67,7 @@ function DropDownMenu({
         onClose={handleClose}
         menuItems={menuItems}
       />
-    </div>
+    </>
   )
 }
 

diff --git a/packages/core/ui/Menu.tsx b/packages/core/ui/Menu.tsx
@@ -405,7 +405,6 @@ function Menu(props: MenuProps) {
 
   return (
     <Popover
-      transitionDuration={0}
       open={open}
       onClose={onClose}
       BackdropProps={{ invisible: true }}

diff --git a/packages/core/ui/theme.ts b/packages/core/ui/theme.ts
@@ -128,6 +128,16 @@ export function createJBrowseDefaultProps(/* palette: PaletteOptions = {} */) {
           size: 'small' as const,
         },
       },
+      MuiPopover: {
+        defaultProps: {
+          transitionDuration: 0,
+        },
+      },
+      MuiMenu: {
+        defaultProps: {
+          transitionDuration: 0,
+        },
+      },
       MuiMenuList: {
         defaultProps: {
           dense: true,