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

fix: secondary items not visible on IE11 #21

Closed
wants to merge 1 commit into from

Conversation

jonirrings
Copy link

缘起于rc-menu的 react-component/menu#444
调试后发现,本repo的example-base,在IE11下,非第一个item不会展现出来。
参考 #2 ,得到最小化修正rc-menu的方法。

经本地测试,可解决 react-component/menu#444

因对相关组理解不够深刻,关联影响未知。

@vercel
Copy link

vercel bot commented May 9, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
overflow ✅ Ready (Inspect) Visit Preview May 9, 2022 at 9:57AM (UTC)

@vercel
Copy link

vercel bot commented May 9, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
overflow 🔄 Building (Inspect) May 9, 2022 at 9:53AM (UTC)

@@ -65,7 +65,7 @@ function InternalItem<ItemType>(
if (!invalidate) {
overflowStyle = {
opacity: mergedHidden ? 0 : 1,
height: mergedHidden ? 0 : UNDEFINED,
// height: mergedHidden ? 0 : UNDEFINED,
Copy link
Member

Choose a reason for hiding this comment

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

不能直接删,这个会影响现有功能的。

Copy link
Author

Choose a reason for hiding this comment

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

ok,我找找绕过的方法。

@afc163
Copy link
Member

afc163 commented May 9, 2022

要找到 IE11 为啥出不来的原因。

@jonirrings
Copy link
Author

弄出最小重现bug的demo了……
bug不在这边,先关掉这个pr

@jonirrings jonirrings closed this May 10, 2022
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.

2 participants