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(projects): text level low. #409 #410

Closed
wants to merge 2 commits into from

Conversation

alleycharming
Copy link
Contributor

Close #409

@honghuangdc
Copy link
Member

button-icon 这个组件用到了很多地方,之前也改过,所以还是多测试下,各种布局下的展示以及移动端的展示

@alleycharming
Copy link
Contributor Author

iPad Mini success
iPad Air success
iPad Pro success
Surface Pro 7 success
Asus Zenbook Fold success
Nest Hub success
Nest Hub Max success

FullScreen 在平板下 都不展示
GlobalSearch 手机端内容铺满全屏不展示
但是在平板下 需要将tooltip-placement置为 left

@honghuangdc
Copy link
Member

好的,辛苦了,我先验证下

@@ -25,7 +25,7 @@ const props = withDefaults(defineProps<Props>(), {
icon: '',
tooltipContent: '',
tooltipPlacement: 'bottom',
zIndex: 98
zIndex: 99
Copy link
Member

Choose a reason for hiding this comment

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

我有印象这里专门加个 zIndex 属性去控制不同地方的层级,所以你这里不应该改属性的默认值

去这个文件 src/layouts/modules/global-menu/first-level-menu.vue 给组件MenuToggler 加个zIndex: 99

@honghuangdc
Copy link
Member

好的,辛苦了,你把这个PR关闭点,然后创建一个commit来进行PR

@alleycharming
Copy link
Contributor Author

非常感谢耐心指导. 感谢项目 有了可以参与并提交 pr 的机会

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.

[Bug]: 左侧菜单混合模式,收缩折叠图标的文本提示层级低
2 participants