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

Exporting with auto bounds doesn't account for outer stroke #2129

Open
paulwrath1223 opened this issue Dec 6, 2024 · 3 comments
Open

Exporting with auto bounds doesn't account for outer stroke #2129

paulwrath1223 opened this issue Dec 6, 2024 · 3 comments
Labels
Feature New feature or request Paper Cut A small UX annoyance we should strive to improve

Comments

@paulwrath1223
Copy link
Contributor

When exporting an SVG with transperancy, the bounding box doesn't seem to include the outer stroke weight. So if you export a square with a stroke width of 8 px, the exported image will have the outer stroke cut off.

This is also the case in the dev deployment.

The exported svg: (notice how the borders are visibly cut off)
Spelling bee cell

The project file (renamed to have .txt so github lets me upload it):
Spelling bee cell.graphite.txt

Changing the SVG viewbox, width, height, and remoxing the transformation to restore the boundaries to those present in the project (1000px^2 for me) fixed the issue completely.

@0HyperCube
Copy link
Member

Duplicate of #2113.

@0HyperCube 0HyperCube closed this as not planned Won't fix, can't repro, duplicate, stale Dec 7, 2024
@github-project-automation github-project-automation bot moved this from Short-Term to Completed This Milestone in Task Board Dec 7, 2024
@Keavon Keavon removed this from Task Board Dec 7, 2024
@Keavon
Copy link
Member

Keavon commented Dec 24, 2024

I'm actually going to make #2113 a duplicate of this, even though this is more recent, because it more clearly describes the problem.

@Keavon Keavon reopened this Dec 24, 2024
@Keavon Keavon added Feature New feature or request Rust Paper Cut A small UX annoyance we should strive to improve labels Dec 24, 2024
@Keavon Keavon changed the title SVG export bounding box incorrect. SVG export bounding box doesn't account for outer stroke Dec 24, 2024
@github-project-automation github-project-automation bot moved this to Short-Term in Task Board Dec 24, 2024
@paulwrath1223
Copy link
Contributor Author

Apologies for the duplicate, I tried looking for SVG errors, not realizing the issue applied to other formats too (as described by #2113) It's really not that big of a deal for me, but I figured you could decide if it's worth fixing.

@Keavon Keavon changed the title SVG export bounding box doesn't account for outer stroke Exporting with auto bounds doesn't account for outer stroke Dec 24, 2024
@Keavon Keavon removed the Rust label Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request Paper Cut A small UX annoyance we should strive to improve
Projects
Status: Short-Term
Development

No branches or pull requests

3 participants