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

Overflow menu item key duplicates #131

Closed
embeoo opened this issue Nov 9, 2021 · 5 comments
Closed

Overflow menu item key duplicates #131

embeoo opened this issue Nov 9, 2021 · 5 comments

Comments

@embeoo
Copy link

embeoo commented Nov 9, 2021

I've got an issue with console warnings for duplicate keys on the overflow menu items.
I can't reproduce in your example (no react?) but in this unrelated sandbox:
https://codesandbox.io/s/0l2oc

1 Open devtools/console
2 Click to add one tab
3 Shrink view to get overflow menu button
4 Click overflow menu button to view menu (no need to select anything)
5 Shrink view to be smaller than 1 tab's width

Console will be full of 'Encountered two children with the same key, tab1' warnings.

@embeoo
Copy link
Author

embeoo commented Nov 12, 2021

And I thought it was so simple and clear..
With shrinking view, I mean making the white browser emulator pane less wide.Attached an image of the end result.
Is there anything about the reproduction you think is more unclear / the reason you can't reproduce?
Screenshot 2021-11-12 at 14 14 50

@rinick
Copy link
Member

rinick commented Nov 12, 2021

It's an issue in rc-tabs lib, I created a ticket there

react-component/tabs#444

@rinick rinick closed this as completed Nov 12, 2021
@embeoo
Copy link
Author

embeoo commented Jan 17, 2022

@rinick Hi, would you consider making a pull request with the change you proposed in the rc-tabs issue above?

@rinick
Copy link
Member

rinick commented Jan 18, 2022

@rinick Hi, would you consider making a pull request with the change you proposed in the rc-tabs issue above?

I did. just check the links in the above issue.

If you really need this PR approved ASAP, you can try these

  1. create a test case to reproduce it without rc-dock, maybe with only rc-tabs, or with antd tabs component
  2. find a way to increase the code coverage for the PR.

If that's too hard, you can use patch-package to fix that issue locally
https://www.npmjs.com/package/patch-package

@embeoo
Copy link
Author

embeoo commented Jan 18, 2022

My bad, I did not check, thought it stalled after the welcome of a PR.

Thanks, I'll see if I can contribute.

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

No branches or pull requests

2 participants