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 (#19221) (CP: 23.5) #19242

Merged
merged 2 commits into from
Apr 25, 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

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
Copy link

github-actions bot commented Apr 24, 2024

Test Results

  960 files  ±0    960 suites  ±0   45m 9s ⏱️ +15s
6 285 tests +3  6 236 ✅ +3  49 💤 ±0  0 ❌ ±0 
6 550 runs  +6  6 493 ✅ +6  57 💤 ±0  0 ❌ ±0 

Results for commit 3253259. ± Comparison against base commit c8154de.

♻️ This comment has been updated with latest results.

Copy link

sonarcloud bot commented Apr 24, 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

@tltv tltv merged commit 1a7635d into 23.5 Apr 25, 2024
26 checks passed
@tltv tltv deleted the cherry/cherrypick-19221-to-23.5 branch April 25, 2024 07:25
ZheSun88 pushed a commit that referenced this pull request Apr 25, 2024
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
ZheSun88 pushed a commit that referenced this pull request Apr 25, 2024
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
vaadin-bot added a commit that referenced this pull request Apr 25, 2024
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

Co-authored-by: Marco Collovati <[email protected]>
mshabarov pushed a commit that referenced this pull request Apr 26, 2024
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

Co-authored-by: Marco Collovati <[email protected]>
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 23.5.1.

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

Successfully merging this pull request may close these issues.

3 participants