【问题标题】:Clicking same item closes menu but different item keeps menu open单击相同的项目关闭菜单,但不同的项目保持菜单打开
【发布时间】:2018-03-21 04:02:24
【问题描述】:

我已经建立了一个带有图标的侧边菜单,单击每个图标后,一个菜单会滑出,上面有相应的项目。通过使用

const handleCloseWhenOpen = menuOpen ? { onClick: this.closeMenu } : null;

然后放

{...handleCloseWhenOpen}

作为一个道具...这使得当我单击 icon1 时菜单打开,再次单击 icon1 时菜单关闭。如果我单击 icon1,菜单会打开,然后单击 icon2,而不是仅仅切换菜单数据,菜单会关闭,我必须再次单击 icon2 以获取包含 icon2 数据的菜单。我只希望在单击以打开它的 same 图标再次单击时关闭菜单。

任何帮助都会很棒。

export default class SidebarNav extends Component {
  static propTypes = {
    children: PropTypes.any,
  };
  constructor() {
    super();
    this.state = {
      menuOpen: false,
      sideBarItems: new List(),
      isSelected: false,
    };
  }

  openMenu = (value) => {
    let menuData = [];
    switch (value) {
      case 'icon1':
        menuData = Data.icon1Data;
        break;
      case 'icon2':
        menuData = Data.icon2Data;
    }
    this.setState({
      menuOpen: true,
      sideBarItems: menuData,
      selectedIcon: value,
    });
  }

  closeMenu = (value) => {
    this.setState({
      menuOpen: false,
      selectedIcon: value,
    });
  }

  render() {
    const {
            menuOpen,
            sideBarItems,
            selectedIcon,
          } = this.state;
    const { children } = this.props;
    const handleCloseWhenOpen = menuOpen ? { onClick: this.closeMenu } : null;
    return (
    <PageWrapper>
      <SideNav
        onClick={this.openMenu}
        selectedIcon={selectedIcon}
      />
      <PageContentWrapper>
        <Sidebar.Pushable as={Segment}>
          <PushMenu
            visible={menuOpen}
            sideBarItems={sideBarItems}
            {...handleCloseWhenOpen}
          />
          <Sidebar.Pusher {...handleCloseWhenOpen}>
            {children}
          </Sidebar.Pusher>
        </Sidebar.Pushable>
      </PageContentWrapper>
    </PageWrapper>
    );
  }
}

export default class SideNav extends Component {
  static propTypes = {
    onClick: PropTypes.func,
    selectedIcon: PropTypes.any,
  }
  render() {
    const {
            onClick,
            selectedIcon,
          } = this.props;
    return (
    <SideBarDiv>
        <IconList>
        {
          Data.icon1Data.map((item, itemIndex) => {
            return (
              <IconWrapper
                key={itemIndex}
                className={
                selectedIcon === item.get('value')
                ? 'Icon-pressed-shadow'
                : 'Icon-right-shadow'
                }
                onClick={() => onClick(item.get('value'))}
              >
                  <ListItem>
                    <Icon name={item.get('name')} />
                      </ListItem>
              </IconWrapper>
            );
          })
        }
        {
          Data.icon2Data.map((item, itemIndex) => {
            return (
              <IconWrapper
                key={itemIndex}
                className={
                selectedIcon === item.get('value')
                ? 'Icon-pressed-shadow'
                : 'Icon-right-shadow'
                }
                onClick={() => onClick(item.get('value'))}
              >
                <ListItem>
                  <Icon name={item.get('name')} />
                  <NotificationLabel>3</NotificationLabel>
                </ListItem>
              </IconWrapper>
            );
          })
        }
      </IconList>
    </SideBarDiv>
    );
  }
}

export default class PushMenu extends Component {
  constructor() {
    super();
    this.state = {
      visible: false,
    };
  }
  static propTypes = {
    visible: PropTypes.bool,
    sideBarItems: PropTypes.instanceOf(Immutable.List),
  }
  render() {
    const {
            sideBarItems,
            visible,
          } = this.props;
    return (
      <Sidebar
        className='Push-menu'
        animation='push'
        width='thin'
        visible={visible}
      >
      <div>
          {
            sideBarItems ? sideBarItems.map((menuTitle, menuTitleIndex) => {
              return (
                <div key={menuTitleIndex}>
                <Header>{menuTitle.get('title')}</Header>
                <Linebreak />
              </div>
              );
            })
            : <Header>Content</Header>
          }
        </div>
        <List>
          {
            sideBarItems ? sideBarItems.map((menuItem, menuItemindex) => {
              return (
                <li key={menuItemindex}>
                    {
                    !!menuItem.get('childItems') &&
                    menuItem.get('childItems').map((childItem) => {
                      return (
                        <div key={childItem.get('name')}>
                          <ListItemHeader>{childItem.get('name')}</ListItemHeader>
                          <List>
                            {
                              childItem.get('nestedItems').map((nestedItem) => {
                                return (
                                  <ListLink key={nestedItem.get('name')}>
                                    <Link
                                      activeStyle={{ fontWeight: 'bold' }}
                                      to={nestedItem.get('route')}
                                    >
                                      <li>{nestedItem.get('name')}</li>
                                    </Link>
                                </ListLink>
                                  );
                              })
                            }
                          </List>
                        </div>
                      );
                    })
                  }
                </li>
              );
            })
            : null
          }
        </List>
          </Sidebar>
    );
  }
}

【问题讨论】:

    标签: javascript reactjs toggle state


    【解决方案1】:

    SidebarNav 中的状态有一个字段menuOpen,用于跟踪 any 菜单是否打开。这是您运行handleCloseWhenOpen 时检查的内容。 因此,当您单击 Icon1 时,menuOpen 设置为 true。接下来,当您单击 Icon2 menuOpen 时,会检查并发现为 true 并切换为 false。因此,单击 Icon2 会关闭 Icon1 的菜单,而不是打开 Icon2 的菜单。

    要纠正这种行为,您需要做的是分别跟踪每个图标菜单的状态。因此,如果您想跟踪 SidebarNav 中的状态,您可以在状态对象中为每个图标设置一个字段,例如

    this.state={
        .....
        menu1Open:false,
        menu2Open:false,
        .....
    }
    

    然后检查您需要切换的字段,具体取决于单击的图标;并相应地渲染菜单。

    如果您认为这会变得复杂,您可以让每个SideNav 跟踪自己的状态,将菜单数据作为道具传递给图标,并让点击操作切换菜单的打开和关闭。

    【讨论】:

    • “每个图标的字段”是什么意思?像道具?
    • 这是有道理的。谢谢你。这就是答案。我很感激。
    • 如果我认为它会变得混乱,我认为它会因为有 12 个图标/菜单......第二个选项是让 SideNav 跟踪它自己的状态。所以我只是把 SideNav 组件中的构造函数做对了吗?我已经编辑了原始帖子以包含 PushMenu 组件。
    猜你喜欢
    • 1970-01-01
    • 2014-03-25
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2012-08-30
    • 1970-01-01
    • 1970-01-01
    • 2019-08-31
    相关资源
    最近更新 更多