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

Package release 3.4.0 #183

Merged
merged 7 commits into from
Aug 21, 2024
Merged

Package release 3.4.0 #183

merged 7 commits into from
Aug 21, 2024

Conversation

IDisposable
Copy link
Member

Fixes for ShadowSlotElement nodes with no assignedNodes to fall back to childNodes. Also fixed some more node.js compatibility (fixes #148 thanks @cWenyu).
Added option to not copy over the web fonts when generating an SVG to allow smaller/faster conversions (fixes #158 thanks @davidburns573 and @YujiaCheng1996).
Updated spec images to match current Chrome output.
Added code to unit-test so emitted debug string when there's a SVG output it is properly escaped.
Bumped dev dependencies for mocha 10.7.3 and eslint 9.9.0.

As they fix bugs, my spec creeps.
Need to escape the emitted SVG in the debug output so we can just save as HTML and see how things looked different.
This will (optionally) reduce the size of the SVG and speed things up when no fonts need to be copied.
Thanks to @davidburns573 and @YujiaCheng1996
Fixes #158
When the element's  has no assignedNodes, fall-back to the node's childNodes for ShadowSlotElements.
Thanks @cWenyu
Fixes #178, Merges #179
@IDisposable IDisposable merged commit 68e143a into main Aug 21, 2024
3 checks passed
@IDisposable IDisposable deleted the release-3-4-0 branch August 21, 2024 23:48
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.

Need embed fonts to be optional make it posible to shift
1 participant