-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Incorrect Popover z-index in Edit Variant modal #8158
Labels
Comments
Thanks for opening this issue @Dominic-Preap - would you be up for working on a PR to fix this? |
@srindom sure, I'll check contribution guide and make a PR. |
@Dominic-Preap - how did it go with getting started on this? Happy to help if you are bumping into challenges? |
I'm so sorry, I've been busy lately 😔
…On Mon, Jul 22, 2024, 4:25 PM Sebastian Rindom ***@***.***> wrote:
@Dominic-Preap <https://github.com/Dominic-Preap> - any luck?
—
Reply to this email directly, view it on GitHub
<#8158 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADQ5Z6RZT3SOEPY3OMNJ4RLZNTF2DAVCNFSM6AAAAABLAAQG4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBSGQ4TQOBVGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hey! |
Fixed by @adevinwild in #8358 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Bug report
Describe the bug
Popover of Edit Variant is behind number input.
System information
Medusa version (including plugins):
@medusajs/admin": "^7.1.14"
Node.js version: 20.9.0
Database: Postgresql
Operating system: Windows 10
Browser (if relevant): Chrome
Steps to reproduce the behavior
Expected behavior
Popover should be in front of input.
Screenshots
Code snippets
N/A
Additional context
N/A
The text was updated successfully, but these errors were encountered: