From 8848fccb285fdcd373da80b7d938d87767e8a0e7 Mon Sep 17 00:00:00 2001 From: "Andrew C. Dvorak" Date: Mon, 22 Jan 2018 17:03:54 -0800 Subject: [PATCH] fix(demos): Fix drawer menu icon position in RTL (#1931) Steps to repro: 1. View the [Persistent Drawer](https://material-components-web.appspot.com/drawer/persistent-drawer.html) and [Temporary Drawer](https://material-components-web.appspot.com/drawer/temporary-drawer.html) demo pages in Edge 2. Open the F12 dev tools 3. Run this snippet in the console to toggle RTL: ```js var el = document.documentElement; el.dir = el.dir === 'rtl' ? '' : 'rtl'; ``` 4. Observe that the horizontal alignment of the toolbar menu icon is incorrect in RTL mode Below is a screenshot of the bug this PR is fixing. - Left side: IE 11 (correct behavior) - Right side: MS Edge (buggy positioning) ![Screenshot of RTL drawer demo in IE 11 and Edge](https://user-images.githubusercontent.com/409245/34791933-50e44d48-f5fb-11e7-936b-ae3c3a016d85.png) --- demos/drawer/persistent-drawer.html | 2 +- demos/drawer/temporary-drawer.html | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/demos/drawer/persistent-drawer.html b/demos/drawer/persistent-drawer.html index f9c52b9a730..22ebd6833bc 100644 --- a/demos/drawer/persistent-drawer.html +++ b/demos/drawer/persistent-drawer.html @@ -98,7 +98,7 @@
- + diff --git a/demos/drawer/temporary-drawer.html b/demos/drawer/temporary-drawer.html index 63f586dda57..e8a3d411e91 100644 --- a/demos/drawer/temporary-drawer.html +++ b/demos/drawer/temporary-drawer.html @@ -44,7 +44,7 @@
- +