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.

Sign up for GitHub

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

Jump to bottom

<Table> doesn't support old browsers. #1685

Closed
zsxsoft opened this issue Sep 20, 2015 · 4 comments
Closed

<Table> doesn't support old browsers. #1685

zsxsoft opened this issue Sep 20, 2015 · 4 comments
Labels
component: table This is the name of the generic UI component, not the React module!

Comments

@zsxsoft
Copy link

zsxsoft commented Sep 20, 2015

I want to build a Cordova app with Material-UI. But I found that I can't open any page that include <Table> in some devices. So I used an older version of Chrome to test it.
Here is the User-agent.
Mozilla/5.0 (Windows NT 5.1) AppleWebKit/536.11 (KHTML, like Gecko) Chrome/20.0.1132.11 TaoBrowser/3.5 Safari/536.11

Everything is OK if I didn't open the Table.
image

If I clicked on Table, there will be no response. But I can't open other Components page now.
image

Sometimes it will throw some errors like this:
image

However, there're still many mobiles running Android 4.x or 5.0 with old WebView. So..
@zsxsoft
Copy link
Author

zsxsoft commented Sep 20, 2015

image
It's my code here.

@goldbergyoni
Copy link

Same here, also not working from the demo site in IE11:
http://material-ui.com/#/components/table

@jkruder
Copy link
Contributor

jkruder commented Sep 23, 2015

All, see babel's caveats. For an immediate fix, include the babel polyfill like so. A more permanent fix to this problem is to replace for..ins with standard for loops so consumers are not required to utilize a polyfill.

@zsxsoft
Copy link
Author

zsxsoft commented Sep 26, 2015

@jkruder Aha, thanks.

@zsxsoft zsxsoft closed this as completed Sep 26, 2015
mnajdova pushed a commit to mnajdova/material-ui that referenced this issue Nov 10, 2020
@dependabot-preview
Bump cypress-react-unit-test from 2.4.3 to 3.3.1 (mui#1685)
341171f 
Bumps [cypress-react-unit-test](https://github.com/bahmutov/cypress-react-unit-test) from 2.4.3 to 3.3.1.
- [Release notes](https://github.com/bahmutov/cypress-react-unit-test/releases)
- [Commits](cypress-io/cypress-react-unit-test@v2.4.3...v3.3.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
@oliviertassinari oliviertassinari added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Dec 25, 2022
@zannager zannager added component: table This is the name of the generic UI component, not the React module! and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: table This is the name of the generic UI component, not the React module!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jkruder @zsxsoft @oliviertassinari @goldbergyoni @zannager