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

fix: prevent embedded styles to leak main document (#19274) (CP: 24.3) #19293

Merged
merged 2 commits into from
May 3, 2024

Conversation

mcollovati
Copy link
Collaborator

When using an exported a themed Flow web-component, Lumo style may leak the embedding document, causing invalid CSS rules to be applied. This change prevents applying Lumo global imports when the theme is applied to a web-component.

Fixes #12704
Fixes #19265

When using an exported a themed Flow web-component, Lumo style may leak
the embedding document, causing invalid CSS rules to be applied.
This change prevents applying Lumo global imports when the theme is
applied to a web-component.

Fixes #12704
Fixes #19265
Copy link

sonarcloud bot commented May 2, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

github-actions bot commented May 2, 2024

Test Results

1 052 files  ± 0  1 052 suites  ±0   1h 13m 57s ⏱️ - 1m 29s
6 807 tests ± 0  6 762 ✅ ± 0  45 💤 ±0  0 ❌ ±0 
7 128 runs  +38  7 072 ✅ +38  56 💤 ±0  0 ❌ ±0 

Results for commit 24f9038. ± Comparison against base commit c363f22.

@mshabarov mshabarov requested a review from tltv May 2, 2024 10:44
@mcollovati mcollovati merged commit cc1e1be into 24.3 May 3, 2024
26 checks passed
@mcollovati mcollovati deleted the cherry/cherrypick-19274-to-24.3 branch May 3, 2024 07:19
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.3.11.

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

Successfully merging this pull request may close these issues.

None yet

3 participants