Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

封装 Menu 子组件时导致的 key 不能正常传递的坑

オープン
#833 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
38/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
react, typescript
領域
frontend

調査の方向性

まずラップされた SubMenu の例を再現し、src/utils/commonUtil.ts の 4 行目付近を読んでください。次に SubMenuProps を調べて、eventKey がどのように注入されるかを追跡してください。issue に明確化、ドキュメント化、またはリファクタリングが必要かどうかを判断する前に、既存の menu API と関連する実装を確認してください。完了とは、キーの動作とサポートされる使用方法が明示的に解決されていることを意味します。

索引モデルが issue の本文から書いたものです。

説明

这段代码中无论是 custom-sub-menu 还是 custom-sub-menu-inside 都没能正确的传给 Menu.SubMenu 组件,导致不能正常工作:

function CustomSubMenu(props) {
  return (
    <Menu.SubMenu key="custom-sub-menu-inside" title={<div>title</div>}>
      <Menu.Item key="item">item</Menu.Item>
    </Menu.SubMenu>
  );
}

<Menu mode="inline">
  <CustomSubMenu key="custom-sub-menu" />
</Menu>

我查了相关文档也没有找到类似问题,后面我读了源码才看到是在 https://github.com/react-component/menu/blob/master/src/utils/commonUtil.ts#L4 parseChildren 里通过 cloneElement 的方式注入了 props.eventKey 去传递的,而这个 eventKey 也没有声明在 SubMenuProps 类型里,上面的代码修复后应该是:

function CustomSubMenu(props) {
  return (
    <Menu.SubMenu eventKey="custom-sub-menu-inside" title={<div>title</div>}>
      <Menu.Item key="item">item</Menu.Item>
    </Menu.SubMenu>
  );
}

<Menu mode="inline">
  <CustomSubMenu key="custom-sub-menu" />
</Menu>

尽管现在 antdv6 已经弃用 children 但是如果有直接使用 rc-component/menu 或者老版本 antd 项目中依然可能出现此类问题,这里我提交一个 issue 希望可以给后面遇到同样问题的人解惑。

同时我对 https://github.com/react-component/menu/blob/master/src/utils/commonUtil.ts#L4 这里的实现表示不解,能否用更好的方式 refactor(比如 Context)。

Image
主要言語
TypeScript
スター
701
フォーク
266
平均マージ
1日 2時間
マージ済み PR(30日)
2

環境構築

このプロジェクトの環境構築ファイルはまだ確認していません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

react-component/menu のほかの issue

react-component/menu の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。