Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[docs] fix errors on IE11 caused by dr-ui dependencies #8544

Merged
merged 2 commits into from
Jul 24, 2019

Conversation

katydecorah
Copy link
Contributor

Earlier this week the docs team received a report from the GL JS team that they were unable to test the local site on IE11. In addition, IE11 would not load maps on IE11 and showed the webgl is unsupported note despite webgl being enabled.

The bug is directly related to dependencies of dr-ui's Search component which use es2015 features and were not properly polyfilled.

This PR treats the symptoms of this bug in two ways:

  1. Adds mimic-fn and debounce-fn to batfish config's babelInclude to polyfill those dependencies which caused the initial error.
  2. Updates dr-ui with a patch that will prevent IE11 from loading the search component. Since IE users account for less than 1% of traffic and the search framework we are using is not optimized for IE, we decided that graceful degradation is the best option.

image
website now loading properly in IE11

Launch Checklist

  • briefly describe the changes in this PR

Copy link
Contributor

@ryanhamley ryanhamley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this on IE11 and it works now! Thanks for the fix @katydecorah

@katydecorah katydecorah merged commit 8530dc9 into publisher-production Jul 24, 2019
@katydecorah katydecorah deleted the 07-22-debug branch July 24, 2019 19:31
ahk pushed a commit that referenced this pull request Jul 26, 2019
* Update batfish.config.js

* update dr-ui
mourner added a commit that referenced this pull request Jul 30, 2019
* origin/publisher-production:
  update version to v1.2.0 (#8562)
  Fix usage of String.includes which is es6 but not transformed by buble (#8565) (#8573)
  [docs] fix missing location prop to feedback component (#8554)
  Fix typo
  [docs] update dr-ui (#8548)
  [docs] fix errors on IE11 caused by dr-ui dependencies (#8544)
  updates directions plugin version (#8536)
  add dds sdk support
  squash (#8529)
  [docs] adds feedback component (#8507)
mourner pushed a commit that referenced this pull request Jul 30, 2019
* Update batfish.config.js

* update dr-ui
mourner pushed a commit that referenced this pull request Jul 30, 2019
* Update batfish.config.js

* update dr-ui
mourner pushed a commit that referenced this pull request Jul 30, 2019
* Update batfish.config.js

* update dr-ui
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants