Skip to content

Conversation

@rob0rt
Copy link
Contributor

@rob0rt rob0rt commented Mar 26, 2015

This PR is a merge of Master into css-in-js, as well as an overhaul of the docs, converting the components into new ES5 Classes as specified in React 0.13.

Other small changes include general code cleanliness, such as converting things like:

var AppBar = mui.AppBar;
var AppCanvas = mui.AppCanvas;
var Menu = mui.Menu;
var IconButton = mui.IconButton;

to:

var { AppBar, AppCanvas, Menu, IconButton } = mui;

As well as whitespace/ newlines (automatically fixed by my IDE).

I did not convert date-picker, menus, and text-fields, as those seemed broken at the current state of the tree so I could not test to make sure I didn't break them.

@rob0rt
Copy link
Contributor Author

rob0rt commented Mar 26, 2015

I also deliberately opened this against css-in-js since I know that's where a lot of work is taking place; I can open it against master if that's better suited.

mmrtnz added a commit that referenced this pull request Mar 28, 2015
Merge from master and React 0.13/ES6 changes
@mmrtnz mmrtnz merged commit 24db37b into mui:css-in-js Mar 28, 2015
@mmrtnz
Copy link
Contributor

mmrtnz commented Mar 28, 2015

Thanks @BobertForever! Yes css-in-js is where all the work is going on. Any PRs to that branch won't be seen until v0.8.0. This was really helpful. I can take a look at what's breaking those components and have them updated to React v0.13.

@rob0rt rob0rt deleted the css-in-js branch March 29, 2015 03:11
@rob0rt rob0rt restored the css-in-js branch March 29, 2015 03:11
@zannager zannager added the scope: system The system, the design tokens / styling foundations used across components. eg. @mui/system with MUI label Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: system The system, the design tokens / styling foundations used across components. eg. @mui/system with MUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants