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

[Lens] turn on veil in the right place #175550

Closed
wants to merge 1 commit into from

Conversation

drewdaemon
Copy link
Contributor

Summary

Fixes a bug where the veil was enabled on embeddable instead of in the editor.

@drewdaemon
Copy link
Contributor Author

/ci

2 similar comments
@drewdaemon
Copy link
Contributor Author

/ci

@drewdaemon
Copy link
Contributor Author

/ci

@stratoula
Copy link
Contributor

Closing in favor of #175691 Drew

@stratoula stratoula closed this Jan 26, 2024
@drewdaemon
Copy link
Contributor Author

/ci

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

stratoula added a commit that referenced this pull request Jan 26, 2024
## Summary

Follow up of #171081

- Address Julia's last comment on the PR
- Fix the veil bug correctly as Drew proposed here
#175550 instead of adding a
variable in the embeddable level as I did in the aforementioned PR. We
dont want the veil in the embeddable level

---------

Co-authored-by: Julia Rechkunova <[email protected]>
CoenWarmer pushed a commit to CoenWarmer/kibana that referenced this pull request Feb 15, 2024
## Summary

Follow up of elastic#171081

- Address Julia's last comment on the PR
- Fix the veil bug correctly as Drew proposed here
elastic#175550 instead of adding a
variable in the embeddable level as I did in the aforementioned PR. We
dont want the veil in the embeddable level

---------

Co-authored-by: Julia Rechkunova <[email protected]>
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.

3 participants