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.

Sign up for GitHub

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

Jump to bottom

[Component] [table] when show-overflow-tooltip is set to true, popper z-index is incorrectly calculated #11895

Closed
Mario34 opened this issue Mar 8, 2023 · 6 comments · Fixed by #11900
Closed

[Component] [table] when show-overflow-tooltip is set to true, popper z-index is incorrectly calculated #11895

Mario34 opened this issue Mar 8, 2023 · 6 comments · Fixed by #11900

Comments

@Mario34
Copy link
Contributor

Mario34 commented Mar 8, 2023 •
edited

Bug Type: Component

Environment

  • Vue Version: 3.2.45
  • Element Plus Version: 2.2.33
  • Browser / OS: Chrome 102.0.5005.61/Mac os 12.2.1
  • Build Tool: Vite

Reproduction

Related Component

  • el-table

Reproduction Link

Element Plus Playground

Steps to reproduce

Switch version 2.2.33 2.3.32, compare tooltip z-index

What is Expected?

The z-index should be calculated based on the configuration

What is actually happening?

The global config z-index is not set correctly

Additional comments

(empty)

There are warnings like this runtime-core.esm-bundler.js:40 [Vue warn]: inject() can only be used inside setup() or functional components., when you hover over the overflow table cell.

image
@Mario34
Copy link
Contributor Author

Mario34 commented Mar 8, 2023

image

@chenxch
Copy link
Member

chenxch commented Mar 8, 2023

Please upgrade to the latest version.

Mario34 added a commit to Mario34/element-plus that referenced this issue Mar 8, 2023
@Mario34
fix(components): [table] useZIndex is called outside setup (element-p…
0fd77c4 
…lus#11895)
@Mario34 Mario34 mentioned this issue Mar 8, 2023
Merged
3 tasks
@Mario34
Copy link
Contributor Author

Mario34 commented Mar 8, 2023

Please upgrade to the latest version.

@chenxch The latest version can also repeat the problem.

@Mario34
Copy link
Contributor Author

Mario34 commented Mar 8, 2023 •
edited

@chenxch Would you mind reviewing the #11900

Mario34 added a commit to Mario34/element-plus that referenced this issue Mar 8, 2023
@Mario34
fix(components): [table] useZIndex is called outside setup (element-p…
87b391e 
…lus#11895)
@Mario34 Mario34 changed the title [Component] [table] 2.2.33 version table show-overflow-tooltip z-index is different from earlier versions [Component] [table] table show-overflow-tooltip z-index is different from earlier versions Mar 8, 2023
@Mario34 Mario34 changed the title [Component] [table] table show-overflow-tooltip z-index is different from earlier versions [Component] [table] when show-overflow-tooltip is set to true, popper z-index is incorrectly calculated Mar 8, 2023
@chenxch
Copy link
Member

chenxch commented Mar 8, 2023

/cc @jw-foss

jw-foss pushed a commit that referenced this issue Mar 8, 2023
@Mario34
fix(components): [table] useZIndex is called outside setup (#11895) (#…
da73213 
…11900)
@jw-foss jw-foss mentioned this issue Mar 8, 2023
Closed
This was referenced Mar 8, 2023
Closed
Merged
loosheng pushed a commit to loosheng/element-plus that referenced this issue Mar 26, 2023
@Mario34 @loosheng
fix(components): [table] useZIndex is called outside setup (element-p…
a5dd685 
…lus#11895) (element-plus#11900)
@github-actions
Copy link

github-actions bot commented Apr 7, 2023

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

此 issue 已被自动锁定，因为关闭后没有任何近期活动。如果有相关 bug，请重新创建一个新 issue。

@github-actions github-actions bot locked and limited conversation to collaborators Apr 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(components): [table] useZIndex is called outside setup (#11895) Mario34/element-plus
2 participants
@chenxch @Mario34