[SwipeableDrawer] Fix prevented inner scroll (mui#22254)

BramKaashoek · web-flow · commit b1b0c62e8cdf · 2020-08-19T01:09:32.000+02:00
diff --git a/packages/material-ui/src/Slider/Slider.test.js b/packages/material-ui/src/Slider/Slider.test.js
@@ -26,7 +26,7 @@ function createTouches(touches) {
 }
 
 describe('<Slider />', () => {
-  // Not support by IE 11
+  // only run in supported browsers
   if (typeof Touch === 'undefined') {
     return;
   }
diff --git a/packages/material-ui/src/StepButton/StepButton.test.js b/packages/material-ui/src/StepButton/StepButton.test.js
@@ -72,6 +72,7 @@ describe('<StepButton />', () => {
 
   describe('event handlers', () => {
     it('should forward mouseenter, mouseleave and touchstart', function touchTests() {
+      // only run in supported browsers
       if (typeof Touch === 'undefined') {
         this.skip();
       }
diff --git a/packages/material-ui/src/SwipeableDrawer/SwipeableDrawer.js b/packages/material-ui/src/SwipeableDrawer/SwipeableDrawer.js
@@ -54,7 +54,7 @@ function getDomTreeShapes(element, rootNode) {
   // Adapted from https://github.com/oliviertassinari/react-swipeable-views/blob/7666de1dba253b896911adf2790ce51467670856/packages/react-swipeable-views/src/SwipeableViews.js#L129
   let domTreeShapes = [];
 
-  while (element && element !== rootNode) {
+  while (element && element !== rootNode.parentElement) {
     const style = ownerWindow(rootNode).getComputedStyle(element);
 
     if (
diff --git a/packages/material-ui/src/SwipeableDrawer/SwipeableDrawer.test.js b/packages/material-ui/src/SwipeableDrawer/SwipeableDrawer.test.js

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ function createTouches(touches) {`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`describe('<Slider />', () => {`
`29`		`- // Not support by IE 11`
	`29`	`+ // only run in supported browsers`
`30`	`30`	`if (typeof Touch === 'undefined') {`
`31`	`31`	`return;`
`32`	`32`	`}`
Original file line number	Diff line number	Diff line change
`@@ -72,6 +72,7 @@ describe('<StepButton />', () => {`
`72`	`72`
`73`	`73`	`describe('event handlers', () => {`
`74`	`74`	`it('should forward mouseenter, mouseleave and touchstart', function touchTests() {`
	`75`	`+ // only run in supported browsers`
`75`	`76`	`if (typeof Touch === 'undefined') {`
`76`	`77`	`this.skip();`
`77`	`78`	`}`