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

[EuiInMemoryTable] inside an [EuiModal] covers it with a red layer #6328

Closed
lior-fortifieddata opened this issue Oct 27, 2022 · 6 comments · Fixed by #6343
Closed

[EuiInMemoryTable] inside an [EuiModal] covers it with a red layer #6328

lior-fortifieddata opened this issue Oct 27, 2022 · 6 comments · Fixed by #6343
Labels

Comments

@lior-fortifieddata
Copy link

https://codesandbox.io/s/sharp-meninsky-v7x6pt?file=/demo.js

see gif

it's coming from here
image

chrome_GjkFns0FYP

@lior-fortifieddata lior-fortifieddata changed the title InMemoryTable inside a modal causes red layer over it InMemoryTable inside a Modal causes red layer over it Oct 27, 2022
@lior-fortifieddata lior-fortifieddata changed the title InMemoryTable inside a Modal causes red layer over it [EuiInMemoryTable] inside an [EuiModal] causes red layer over it Oct 27, 2022
@lior-fortifieddata lior-fortifieddata changed the title [EuiInMemoryTable] inside an [EuiModal] causes red layer over it [EuiInMemoryTable] inside an [EuiModal] covers it with a red layer Oct 27, 2022
@miukimiu
Copy link
Contributor

Hi @lior-fortifieddata.

Thanks for reporting.

This issue is going to be fixed on #6321.

@miukimiu miukimiu added the bug label Oct 27, 2022
@lior-fortifieddata
Copy link
Author

lior-fortifieddata commented Oct 27, 2022 via email

@lior-fortifieddata
Copy link
Author

lior-fortifieddata commented Nov 7, 2022

@breehall @constancecchen @miukimiu
The bug is not fixed on 70.1.0, even though the commit of the related issue is merged. should it?

@cee-chen
Copy link
Member

cee-chen commented Nov 7, 2022

The commit being merged into main does not mean it has been released - this bugfix is not yet in our regular weekly release cadence. We have instead backported it to 67.x for our latest Kibana upgrade PR to ensure it reaches Kibana by 8.6 FF.

@lior-fortifieddata
Copy link
Author

lior-fortifieddata commented Nov 10, 2022 via email

@cee-chen
Copy link
Member

We usually do a release every Tuesday, so things get in relatively quickly once merged into main. You can subscribe to our repo releases to see a changelog of all commits in that release.

This fix in particular should now be in our latest 70.2.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants