[Collapse] Don't consider hidden if collapsedHeight > 0

mui · Feb 12, 2019 · bf011ce · bf011ce
1 parent 59ee3e5
commit bf011ce
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/packages/material-ui/src/Collapse/Collapse.js b/packages/material-ui/src/Collapse/Collapse.js
@@ -21,7 +21,8 @@ export const styles = theme => ({
     height: 'auto',
     overflow: 'visible',
   },
-  /* Styles applied to the container element when the transition has exited. */
+  // eslint-disable-next-line max-len
+  /* Styles applied to the container element when the transition has exited and `collapsedHeight` != 0px. */
   hidden: {
     vibility: 'hidden',
   },
@@ -156,7 +157,7 @@ class Collapse extends React.Component {
       >
         {(state, childProps) => (
           <Component
-            aria-hidden={state === 'exited'}
+            aria-hidden={state === 'exited' && collapsedHeight === '0px'}
             className={clsx(
               classes.container,
               {

diff --git a/pages/api/collapse.md b/pages/api/collapse.md
@@ -39,7 +39,7 @@ This property accepts the following keys:
 |:-----|:------------|
 | <span class="prop-name">container</span> | Styles applied to the container element.
 | <span class="prop-name">entered</span> | Styles applied to the container element when the transition has entered.
-| <span class="prop-name">exited</span> | Styles applied to the container element when the transition has exited.
+| <span class="prop-name">hidden</span> | Styles applied to the container element when the transition has exited and `collapsedHeight` != 0px.
 | <span class="prop-name">wrapper</span> | Styles applied to the outer wrapper element.
 | <span class="prop-name">wrapperInner</span> | Styles applied to the inner wrapper element.