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

React 0.14 support #1770

Merged
merged 4 commits into from
Sep 30, 2015
Merged

Conversation

cgestes
Copy link
Contributor

@cgestes cgestes commented Sep 30, 2015

  • add missing dependencies to the docs project. npm 3 do not install peerDependencies automatically anymore.
  • fixes for react-router 1.0

Only application (that wont be used as npm modules) should declare peerDependencies as real dependencies to bring them to the build.

NPM 3 do not install peerDependencies anymore.
@oliviertassinari
Copy link
Member

These dependences are already in the root package.json. Now, they are duplicated, what's the benefit?

@cgestes
Copy link
Contributor Author

cgestes commented Sep 30, 2015

@oliviertassinari npm 3 do not install peerDependencies anymore.

so to run the doc now you have to install them by hand.

(btw I'am currently fixing react-router on the 0.14 branch)

@oliviertassinari
Copy link
Member

npm 3 do not install peerDependencies anymore.

Ok thanks

@shaurya947
Copy link
Contributor

@cgestes this is complete now right (including all fixes for react-router)?

@cgestes
Copy link
Contributor Author

cgestes commented Sep 30, 2015

I hope so.

@cgestes
Copy link
Contributor Author

cgestes commented Sep 30, 2015

btw the code editor is still broken on the doc project, but I think I saw a PR about so, I didn't touch anything.

@cgestes
Copy link
Contributor Author

cgestes commented Sep 30, 2015

and ... the url are nicer than in master, but the webpack-dev-server is not able to serve them correctly, which means reloading a webpage does not work.

we use localhost:3000/home instead of former localhost:3000/#/home

@shaurya947
Copy link
Contributor

I'll go ahead and merge this. Let's try running the docs site off this branch after I merge this and see what else is broken?

shaurya947 added a commit that referenced this pull request Sep 30, 2015
@shaurya947 shaurya947 merged commit 9922d15 into mui:react-0.14-support Sep 30, 2015
@oliviertassinari
Copy link
Member

we use localhost:3000/home instead of former localhost:3000/#/home

Does this mean that in IE 8 and 9 the page is fully reloaded when changing route?

@cgestes
Copy link
Contributor Author

cgestes commented Sep 30, 2015

Yep that's what the current strategy we use does. It is easy to change. We
were using hask before. We can set the key to false to remove _k in the url

On Wed, Sep 30, 2015, 18:36 Olivier Tassinari notifications@github.com
wrote:

we use localhost:3000/home instead of former localhost:3000/#/home

Does this mean that in IE 8 and 9 the page is fully reloaded when changing
route?


Reply to this email directly or view it on GitHub
#1770 (comment)
.

mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Nov 10, 2020
* Support empty dates for the DatePicker

* Support empty dates for TimePicker

* Support empty dates for mobile toobars

* Support empty dates in YearSelection

* Introduce `toolbarPlaceholder` prop for customization of empty date text

* Add tests

* Do not dispatch `changeFocusedDay` on each mount

* Fix crashes when rendering empty date with luxon and moment

* [TimePicker] Do not throw validation error if value={null}

* Update props description for TimePicker validation props

* Refactor ternary operator to aviod negatiation

* Remove `defaultHighlight` prop
@zannager zannager added the docs Improvements or additions to the documentation label Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants