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

Legacy Context API / findDomNode usage #91

Closed
4 tasks done
Wintereise opened this issue May 3, 2019 · 7 comments
Closed
4 tasks done

Legacy Context API / findDomNode usage #91

Wintereise opened this issue May 3, 2019 · 7 comments

Comments

@Wintereise
Copy link

Wintereise commented May 3, 2019

Prerequisites

Please answer the following questions for yourself before submitting an issue.

  • I am running the latest version
  • I checked the documentation and found no answer
  • I checked to make sure that this issue has not already been filed
  • I'm reporting the issue to the correct repository (for multi-repository projects)

I'm a material dashboard react pro subscriber with current premium support.

The theme (thanks to material-ui 3.x usage) makes use of the legacy context API (which has been marked as deprecated) and findDomNode, both of which cause problems if one turns on React's StrictMode and are marked for removal sometime in the future.

material-ui is going to be fixing these in their 4.x release, will this theme be seeing that update?

Relevant MUI issues.

  1. [RFC] Refs in v4 mui/material-ui#14415
  2. Warnings in strict mode mui/material-ui#13394
@einazare
Copy link
Contributor

einazare commented May 3, 2019

Hello again, @Wintereise ,

Yes, we are going to update our products to the v4 of Material-UI. But you have to understand, and I am not trying to be rude here, I am just trying to explain how we work at Creative-Tim - no offense intended, that beside this project, we have 11 other products that we also have to maintain, and also, while doing this we have to work on developing new products.
Since the React Material products are our most downloaded and used, we try to keep them as updated as we can, but once again, we can't neglect our other products.
We expect that after Material-UI releases version 4, we are going to update all of our products in about
1-3 months, and this is because we have 4 Material products and from our knowledge, Material-UI v4 will have a lot of major changes, for example, they are going to drop the usage of jss, and use emotion - this means that we'll also have to change our jss to emotion.

I hope I've asked your question.

I have although one question, you say something about the PRO version in your comment, you also checked the I'm reporting the issue to the correct repository (for multi-repository projects). Why did you open this issue here, and not on the PRO repository?

All the best,
Manu
ReactJS Developer at Creative-Tim.com (http://creative-tim.com/)

Trying to help the open source community:

If you've liked what you saw, help me with a comment, like, share or subscribe.

@crimsonvspurple
Copy link

In the next major update, please develop with React strict mode.

@Wintereise
Copy link
Author

Wintereise commented May 3, 2019

Hey @einazare,

Yes, we are going to update our products to the v4 of Material-UI. But you have to understand, and I am not trying to be rude here, I am just trying to explain how we work at Creative-Tim - no offense intended, that beside this project, we have 11 other products that we also have to maintain, and also, while doing this we have to work on developing new products.

Fair enough, my intention wasn't to ask "why hasn't it been done yet (doing it already makes no sense, mui v4 is beta software)," but simply to understand if CT had plans to do so.

Regarding the challenges, I fully understand. Thank you for providing the timeline.

I have although one question, you say something about the PRO version in your comment, you also checked the I'm reporting the issue to the correct repository (for multi-repository projects). Why did you open this issue here, and not on the PRO repository?

That would be because I infact did not know that mdr had multiple repos. I googled it, found this repo, opened issues, found the other one afterwards ┐(´∀`)┌ヤレヤレ

@einazare
Copy link
Contributor

einazare commented May 3, 2019

In the next major update, please develop with React strict mode.

Ok, so then it will take us more than 1-3 months. I'll discuss this with our board.

@einazare
Copy link
Contributor

einazare commented May 3, 2019

Hello again, @Wintereise ,

That would be because I infact did not know that mdr had multiple repos. I googled it, found this repo, opened issues, found the other one afterwards ┐(´∀`)┌ヤレヤレ

You can get the right repo link from our website (from the comments section on the product page), or from our live docs.

@Flixbox
Copy link

Flixbox commented May 24, 2019

they are going to drop the usage of jss, and use emotion - this means that we'll also have to change our jss to emotion.

Do you have a source on that? According to the blog post We can pretty much still use JSS logic with hooks.

@einazare
Copy link
Contributor

they are going to drop the usage of jss, and use emotion - this means that we'll also have to change our jss to emotion.

Do you have a source on that? According to the blog post We can pretty much still use JSS logic with hooks.

I do not remember where I read that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants