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

ListItem jump #854

Closed
darkleaf opened this issue Jun 15, 2015 · 3 comments
Closed

ListItem jump #854

darkleaf opened this issue Jun 15, 2015 · 3 comments
Labels
component: list This is the name of the generic UI component, not the React module! package: styles Specific to @mui/styles. Legacy package, @material-ui/styled-engine is taking over in v5.

Comments

@darkleaf
Copy link

Hi!
I click on the 'Click and jump' item and it jumps up, but if I click on the checkbox, then all is well.
If I disable overflow property, then item come back.
Item with secondaryText works correct.
I use material-ui 0.9.1 and last Google Chrome 43.

import React from 'react';
import mui, {List, ListItem, Checkbox, TextField} from 'material-ui';

const ThemeManager = new mui.Styles.ThemeManager();

export default React.createClass({
  childContextTypes: {
    muiTheme: React.PropTypes.object
  },
  getChildContext() {
    return {
      muiTheme: ThemeManager.getCurrentTheme()
    };
  },
  render: function() {
    return (
      <div>
        <List>
          <ListItem leftCheckbox={<Checkbox />} secondaryText='without jump'>
            Click and stay
          </ListItem>
          <ListItem leftCheckbox={<Checkbox />}>
            Click and jump
          </ListItem>
        </List>
      </div>
    );
  }
});

_090

@darkleaf
Copy link
Author

Workaround

<ListItem leftCheckbox={<Checkbox />} style={{overflow: 'visible'}}>

@hai-cea
Copy link
Member

hai-cea commented Jun 15, 2015

Thanks for reporting this @darkleaf

@darkleaf
Copy link
Author

Thanks! 👍

@oliviertassinari oliviertassinari added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Dec 25, 2022
@zannager zannager added component: list This is the name of the generic UI component, not the React module! package: styles Specific to @mui/styles. Legacy package, @material-ui/styled-engine is taking over in v5. and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: list This is the name of the generic UI component, not the React module! package: styles Specific to @mui/styles. Legacy package, @material-ui/styled-engine is taking over in v5.
Projects
None yet
Development

No branches or pull requests

4 participants