Merge pull request #2179 from Shopify/safari-13-fix

[Select] change text-rendering to auto to fix Safari bug
Shopify · Sep 23, 2019 · 12bb54d · 12bb54d
2 parents 0bdea25 + 2b6afbd
commit 12bb54d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/UNRELEASED.md b/UNRELEASED.md
@@ -26,6 +26,7 @@ Use [the changelog guidelines](https://git.io/polaris-changelog-guidelines) to f
 - Reverted `Page.primaryAction` forcing `primary` to be `true` ([#2137](https://github.com/Shopify/polaris-react/pull/2137))
 - Removed `React.Children.only` from `AppProvider`and `ThemeProvider` ([#2121](https://github.com/Shopify/polaris-react/pull/2121))
 - Fixed visual bug where button width changed in Filters component ([#1853](https://github.com/Shopify/polaris-react/issues/1853))
+- Changed `text-rendering` to `auto` in `Select` to prevent Safari 13 from crashing ([#2179](https://github.com/Shopify/polaris-react/pull/2179))
 
 ### Documentation
 

diff --git a/src/components/Select/Select.scss b/src/components/Select/Select.scss
@@ -104,6 +104,8 @@ $stacking-order: (
   // Even though the input is invisible, text styles apply to the options menu
   @include text-style-input;
   position: absolute;
+  // Required to solve a bug causing Safari 13 to crash https://bugs.webkit.org/show_bug.cgi?id=202055
+  text-rendering: auto;
   top: 0;
   left: 0;
   z-index: z-index(input, $stacking-order);