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

[Feature Request] [New Component] [] 提个建议:menu菜单严重bug,希望官方大大认真审核PR,不要让一些菜鸡的狗屎代码也能提交上来 #14566

Closed
lixinru1234 opened this issue Oct 17, 2023 · 13 comments · Fixed by #14573

Comments

@lixinru1234
Copy link

Feature type: New Component

Details

  • New component name: 期待朱坤、杨奕等大佬回归

Description

select, popper,还有menu的ul标签的高度设置有问题,无法覆盖掉子元素 这就是导致一旦子元素是透明的就会出现在容器外面,以前是没问题的,menu菜单严重bug,希望官方大大认真审核PR,不要让一些菜鸡的狗屎代码也能提交上了,期待朱坤、杨奕等大佬回归饿了么,现在的代码质量实在是太差劲了 一点测试都没有的 让人觉得像是学生干的活一样 一点严谨性都没有 本来都是用的好好的 每次发个新版本必须带点坑进来

Additional comments

(empty)

image

看了下是这个人写的狗屎代码影响到的,线上的项目pover相关的外层高度全坏了:

image
image

@geedys
Copy link

geedys commented Oct 17, 2023

我也遇到这个问题,看了官方的文档中横向菜单也出了这个bug, 暂时的解决方案:
我在全局样式文件中覆盖了这个:
.el-menu--horizontal{
height: unset;
}

@yj-liuzepeng
Copy link
Contributor

Feature type: New Component

Details

  • New component name: 期待朱坤、杨奕等大佬回归

Description

select, popper,还有menu的ul标签的高度设置有问题,无法覆盖掉子元素 这就是导致一旦子元素是透明的就会出现在容器外面,以前是没问题的,menu菜单严重bug,希望官方大大认真审核PR,不要让一些菜鸡的狗屎代码也能提交上了,期待朱坤、杨奕等大佬回归饿了么,现在的代码质量实在是太差劲了 一点测试都没有的 让人觉得像是学生干的活一样 一点严谨性都没有 本来都是用的好好的 每次发个新版本必须带点坑进来

Additional comments

(empty)

image

看了下是这个人写的狗屎代码影响到的,线上的项目pover相关的外层高度全坏了:

image image

你竟然说这个人是菜鸡,写的还是狗屎,你是多厉害啊!!

@boomboy4
Copy link
Contributor

@YunYouJun 被cue了

@YunYouJun
Copy link
Member

YunYouJun commented Oct 17, 2023

很抱歉给您带来不好的体验,该问题是我的疏忽导致,没有考虑到其他场景且较难通过测试代码发现,我们正在排查原因,将尽快解决该问题。

@lixinru1234
Copy link
Author

很抱歉给您带来不好的体验,该问题是我的疏忽导致,没有考虑到其他场景且较难通过测试代码,我们正在排查原因,将尽快解决该问题。

大佬我错了 我以为是小白提交的 被你合并的

@lixinru1234
Copy link
Author

很抱歉给您带来不好的体验,该问题是我的疏忽导致,没有考虑到其他场景且较难通过测试代码,我们正在排查原因,将尽快解决该问题。

小问题不打紧 大佬注意休息

@likui628
Copy link

likui628 commented Oct 17, 2023

建议保持冷静和尊重,避免使用侮辱性的语言。

@lixinru1234 号是新号,为了骂人特别注册的吧

@baiwusanyu-c
Copy link
Contributor

慕名而来

1 similar comment
@so1ve
Copy link

so1ve commented Oct 17, 2023

慕名而来

@zhangmo8
Copy link

慕名+1

@haoziqaq
Copy link

哈哈哈

@beiranc
Copy link

beiranc commented Oct 17, 2023

慕名而来

@sxzz
Copy link
Collaborator

sxzz commented Oct 17, 2023

本 issue 关闭。
如需继续讨论,请另开 issue 并使用礼貌用语讨论。

@sxzz sxzz closed this as not planned Won't fix, can't repro, duplicate, stale Oct 17, 2023
@element-plus element-plus locked as spam and limited conversation to collaborators Oct 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.