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

Use List/ListItem components from mui #20

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ababol
Copy link
Contributor

@ababol ababol commented Jul 6, 2015

Use List/ListItem components from mui.

We have to use my fork of material-ui but as soon as they have fixed the issue with the utils/extends, we will switch again ;).

Fix #18

@troutowicz
Copy link
Owner

Awesome, I'll take a closer look at this soon!

@@ -4,8 +4,19 @@ import React from 'react';
import ListItem from './ListItem';
import ReactCSSTransitionGroup from './TimeoutTransitionGroup';
import { mergeAndPrefix } from '../utils/stylePropable';
import { List } from 'material-ui';
Copy link
Owner

Choose a reason for hiding this comment

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

import List from 'material-ui/lib/lists/list';

@troutowicz
Copy link
Owner

We still need to figure out why ReactCSSTransitionGroup is broke.

@troutowicz
Copy link
Owner

Another issue I forgot to mention is the ListItem on click ripple color needs to be tweaked.

@ababol
Copy link
Contributor Author

ababol commented Jul 14, 2015

Good point 👍 , I have made the changes.

@ababol
Copy link
Contributor Author

ababol commented Jul 19, 2015

Hey @troutowicz, did you have time to take a look at the ReactCSSTransitionGroup issue?

I will probably investigate further next week.

@troutowicz
Copy link
Owner

I have not, my schedule has been and will be packed through mid this week. Let me know what your investigations turn up. :)

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

Successfully merging this pull request may close these issues.

Use Material-UI's List
2 participants