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

Interactive SVG background: Allow processing whole SVG group for state #2872

Merged
merged 2 commits into from
Nov 16, 2024

Conversation

stefan-hoehn
Copy link
Contributor

Improvement for interactive SVG backgrounds:

Allows to flash and handle state on whole SVG groups without a proxy by using the group's path elements instead, e.g. this whole group can be used directly

svg-groups.mov

Signed-off-by: Stefan Höhn <mail@stefanhoehn.com>
Copy link

relativeci bot commented Nov 16, 2024

#2490 Bundle Size — 10.87MiB (~+0.01%).

8f9ca12(current) vs e3daf94 main#2489(baseline)

Warning

Bundle contains 2 duplicate packages – View duplicate packages

Bundle metrics  Change 2 changes Regression 1 regression
                 Current
#2490
     Baseline
#2489
Regression  Initial JS 1.91MiB(~+0.01%) 1.91MiB
No change  Initial CSS 577.31KiB 577.31KiB
Change  Cache Invalidation 17.55% 18.66%
No change  Chunks 226 226
No change  Assets 249 249
No change  Modules 2940 2940
No change  Duplicate Modules 152 152
No change  Duplicate Code 1.8% 1.8%
No change  Packages 96 96
No change  Duplicate Packages 2 2
Bundle size by type  Change 1 change Regression 1 regression
                 Current
#2490
     Baseline
#2489
Regression  JS 9.08MiB (~+0.01%) 9.08MiB
No change  CSS 864KiB 864KiB
No change  Fonts 526.1KiB 526.1KiB
No change  Media 295.6KiB 295.6KiB
No change  IMG 140.74KiB 140.74KiB
No change  HTML 1.38KiB 1.38KiB
No change  Other 871B 871B

Bundle analysis reportBranch stefan-hoehn:interactive_backgro...Project dashboard


Generated by RelativeCIDocumentationReport issue

Copy link
Contributor

@florian-h05 florian-h05 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few suggestions and a question, otherwise it looks good.

@florian-h05 florian-h05 added enhancement New feature or request main ui Main UI labels Nov 16, 2024
@florian-h05 florian-h05 added this to the 4.3 milestone Nov 16, 2024
Signed-off-by: Stefan Höhn <mail@stefanhoehn.com>
@florian-h05 florian-h05 merged commit e199066 into openhab:main Nov 16, 2024
8 checks passed
@florian-h05 florian-h05 deleted the interactive_background_groups branch November 16, 2024 17:43
@florian-h05 florian-h05 changed the title Interactive SVG background: process whole SVG group for state Interactive SVG background: Allow processing whole SVG group for state Nov 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request main ui Main UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants