Skip to content

feat: add support for dynamic theme in iframes #150

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.

Sign up for GitHub

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

Draft
javier-godoy wants to merge 3 commits into
base: master
Choose a base branch
from
+115 −8

feat: add support for dynamic theme in iframes

37f64d7
Select commit
Loading
Failed to load commit list.
Draft

feat: add support for dynamic theme in iframes #150

feat: add support for dynamic theme in iframes
37f64d7
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 6, 2026 in 15s

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 42 in src/main/java/com/flowingcode/vaadin/addons/demo/ThemeChangeObserver.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this empty statement. 

See more on https://sonarcloud.io/project/issues?id=FlowingCode_CommonsDemo&issues=AZzEBqISTVq4iKB1dYZx&open=AZzEBqISTVq4iKB1dYZx&pullRequest=150

Check warning on line 41 in src/main/java/com/flowingcode/vaadin/addons/demo/ThemeChangeObserver.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Do not forget to remove this deprecated code someday. 

See more on https://sonarcloud.io/project/issues?id=FlowingCode_CommonsDemo&issues=AZzEBqISTVq4iKB1dYZw&open=AZzEBqISTVq4iKB1dYZw&pullRequest=150
View more details on SonarQubeCloud