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

Use mui like global ponyfill #1153

Merged
merged 3 commits into from
Jul 7, 2019
Merged

Use mui like global ponyfill #1153

merged 3 commits into from
Jul 7, 2019

Conversation

TrySound
Copy link
Member

@TrySound TrySound commented Jul 5, 2019

@kof
Copy link
Member

kof commented Jul 7, 2019

TLDR when using global, users webpack creates module wrappers and prevents tree shaking, so we now try to not use global at all.

@kof kof merged commit 743d628 into master Jul 7, 2019
@TrySound TrySound deleted the ponyfill-global branch July 7, 2019 12:03
HenriBeck pushed a commit that referenced this pull request Jul 7, 2019
* master:
  Use componentDidMount for WithStyles (#1157)
  Use mui like global ponyfill (#1153)
  HOC should not attach sheets until mount (#1149)
  v10.0.0-alpha.22
  add react-dom dev dependenncy
  Fix SSR for Hooks based implementation (#1148)
  React-JSS id prop docs and improvements (#1147)
  v10.0.0-alpha.21
  fix changelog
  add support hint to the changelog (#1145)
  Sheets management for css() (#1137)

# Conflicts:
#	packages/react-jss/.size-snapshot.json
bhupinderbola pushed a commit to bhupinderbola/jss that referenced this pull request Sep 17, 2019
* Use mui like global ponyfill

https://twitter.com/IAmTrySound/status/1147174216095211520

With globalThis polyfill we have a browser support problem. Let's try
this instead
https://github.com/mui-org/material-ui/blob/master/packages/material-ui-utils/src/ponyfillGlobal.js

* Rename to globalThis

* changelog
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