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

[Examples] Upgrade for the new format #3078

Merged
merged 1 commit into from
Jan 28, 2016
Merged

[Examples] Upgrade for the new format #3078

merged 1 commit into from
Jan 28, 2016

Conversation

oliviertassinari
Copy link
Member

I have forgotten to take into account the examples in my latest PR to use getMuiTheme (#3076).
I thought it was a good opportunity to clean our examples.
I have:

  • Remove the /components folder
  • Remove the _ for the method handlers
  • Fix the usage of getMuiTheme
  • Upgrade to ES6 classes
  • Use the themeDecorator to provide the muiTheme context.
    I'm convinced that it's should be the only way that we should document it.

@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation PR: Needs Review labels Jan 28, 2016
@@ -1,7 +1,7 @@
import React from 'react';
import ReactDOM from 'react-dom';
import injectTapEventPlugin from 'react-tap-event-plugin';
import Main from './components/main'; // Our custom react component
import Main from './main'; // Our custom react component
Copy link
Member

Choose a reason for hiding this comment

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

Main

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

@alitaheri
Copy link
Member

Thanks a lot @oliviertassinari 👍 👍

I have forgotten to take into account the examples in my lastet PR
to use getMuiTheme.
I though it was a good opportunity to clean our examples.
I have:
- Remove the component folder
- Remove the `_` for the method handlers
- Fix the usage of `getMuiTheme`
- Use the `themeDecorator` to provide the theme.
- Upgraded theme to ES6 classes
I'm convinced that it's should be the only way that we should document
@oliviertassinari
Copy link
Member Author

I'm going ahead 💯.

oliviertassinari added a commit that referenced this pull request Jan 28, 2016
[Examples] Upgrade for the new format
@oliviertassinari oliviertassinari merged commit 6143384 into mui:master Jan 28, 2016
@oliviertassinari oliviertassinari deleted the examples-clean branch January 28, 2016 11:34
@alitaheri
Copy link
Member

Thanks 😁

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.

None yet

2 participants