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

CodeMirror lint tooltip not visible when editor is in modal #19125

Closed
MoonE opened this issue Apr 22, 2024 · 0 comments
Closed

CodeMirror lint tooltip not visible when editor is in modal #19125

MoonE opened this issue Apr 22, 2024 · 0 comments
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface

Comments

@MoonE
Copy link
Contributor

MoonE commented Apr 22, 2024

Describe the bug

.CodeMirror-lint-tooltip has z-index: 200 and is below the modal z-index: 801 thus not visible.

To Reproduce

Steps to reproduce the behavior:

  1. Go to table -> triggers
  2. Click on Add trigger
  3. Insert invalid sql or > 10000 characters to lint icon
  4. Hover over the icon with the mouse
  5. The tooltip is not visible

Expected behavior

The tooltip should be visible above the modal.

Screenshots

image

Server configuration

  • phpMyAdmin version: QA_5_2

Client configuration

  • Browser: Firefox 125.0.2
  • Operating system: linux
@MoonE MoonE mentioned this issue Apr 25, 2024
Merged
@williamdes williamdes added Bug A problem or regression with an existing feature ui Issues relating to the user interface has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete labels Apr 28, 2024
williamdes added a commit that referenced this issue Apr 28, 2024
@williamdes
Add ChangeLog entries for #18240 and #19125
fe8a1f9 
Signed-off-by: William Desportes <williamdes@wdes.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete ui Issues relating to the user interface
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MoonE @williamdes