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

SVGs with <mask> in them broke, affecting the display of several editor icons #82982

Closed
MewPurPur opened this issue Oct 7, 2023 · 4 comments · Fixed by #83281
Closed

SVGs with <mask> in them broke, affecting the display of several editor icons #82982

MewPurPur opened this issue Oct 7, 2023 · 4 comments · Fixed by #83281

Comments

@MewPurPur
Copy link
Contributor

MewPurPur commented Oct 7, 2023

Godot version

Godot v4.2.dev6 - Pop!_OS 22.04 LTS - X11 - Vulkan (Forward+) - integrated Intel(R) Graphics (ADL GT2) () - 12th Gen Intel(R) Core(TM) i5-1235U (12 Threads)

System information

Godot v4.2.dev6 - Pop!_OS 22.04 LTS - X11 - Vulkan (Forward+) - integrated Intel(R) Graphics (ADL GT2) () - 12th Gen Intel(R) Core(TM) i5-1235U (12 Threads)

Issue description

Our SVG icons that use the <mask> element broke.

image

Most likely caused by #82542 but I haven't tested it. Technically, <mask> is not required for these SVGs, but it allows for easy clean cutouts which is why I used them here.

Steps to reproduce

Open Godot and find them

Minimal reproduction project

N/A

@MewPurPur
Copy link
Contributor Author

@capnm

@Calinou Calinou changed the title SVGs with <mask> in them broke SVGs with <mask> in them broke, affecting the display of several editor icons Oct 7, 2023
@capnm
Copy link
Contributor

capnm commented Oct 8, 2023

@MewPurPur Thanks for the report. I can take a look next week. A large PNG before/after image would be helpful.

@MewPurPur
Copy link
Contributor Author

MewPurPur commented Oct 8, 2023

image

From my slightly outdated dev build. (2-3 weeks old)

@capnm
Copy link
Contributor

capnm commented Oct 9, 2023

upscaled-png-diff

capnm added a commit to capnm/godot4 that referenced this issue Oct 13, 2023
https://github.com/thorvg/thorvg/releases/tag/v0.11.1

Godot related:
+ [SwEngine] Enhanced the quality of the dash line corners. godotengine#121
+ [SVG] Fixed a regression bug related to maskContentUnits
	userSpaceOnUse/objectBoundingBox. godotengine#1694

Fixes godotengine#82982
capnm added a commit to capnm/godot4 that referenced this issue Oct 13, 2023
https://github.com/thorvg/thorvg/releases/tag/v0.11.1

Godot related:
+ [SwEngine] Enhanced the quality of the dash line corners. godotengine#121
+ [SVG] Fixed a regression bug related to maskContentUnits
	userSpaceOnUse/objectBoundingBox. godotengine#1694

Fixes godotengine#82982  - icons
Fixes godotengine#83198  - svg ← png/jpg
capnm added a commit to capnm/godot4 that referenced this issue Oct 13, 2023
https://github.com/thorvg/thorvg/releases/tag/v0.11.1

Godot related:
+ [SwEngine] Enhanced the quality of the dash line corners.
+ [SVG] Fixed a regression bug related to maskContentUnit.
	userSpaceOnUse/objectBoundingBox.

Fixes godotengine#82982  - icons
Fixes godotengine#83198  - svg ← png/jpg
capnm added a commit to capnm/godot4 that referenced this issue Oct 15, 2023
https://github.com/thorvg/thorvg/releases/tag/v0.11.1

Godot related:
+ [SwEngine] Enhanced the quality of the dash line corners.
+ [SVG] Fixed a regression bug related to maskContentUnit.
	userSpaceOnUse/objectBoundingBox.

Fixes godotengine#82982  - icons
Fixes godotengine#83198  - svg ← png/jpg
@akien-mga akien-mga added this to the 4.2 milestone Oct 16, 2023
BrianMacIntosh pushed a commit to BrianMacIntosh/godot that referenced this issue Oct 16, 2023
https://github.com/thorvg/thorvg/releases/tag/v0.11.1

Godot related:
+ [SwEngine] Enhanced the quality of the dash line corners.
+ [SVG] Fixed a regression bug related to maskContentUnit,
	userSpaceOnUse/objectBoundingBox.

Fixes godotengine#82982 icons.
Fixes godotengine#83198 svg ← png/jpg.
ProbablyWorks pushed a commit to ProbablyWorks/godot that referenced this issue Oct 22, 2023
https://github.com/thorvg/thorvg/releases/tag/v0.11.1

Godot related:
+ [SwEngine] Enhanced the quality of the dash line corners.
+ [SVG] Fixed a regression bug related to maskContentUnit,
	userSpaceOnUse/objectBoundingBox.

Fixes godotengine#82982 icons.
Fixes godotengine#83198 svg ← png/jpg.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants