Skip to content

New node: Solidify Stroke#1650

Merged
Keavon merged 11 commits into
GraphiteEditor:masterfrom
Kakapio:fill-from-stroke
Mar 12, 2024
Merged

New node: Solidify Stroke#1650
Keavon merged 11 commits into
GraphiteEditor:masterfrom
Kakapio:fill-from-stroke

Conversation

@Kakapio

@Kakapio Kakapio commented Mar 5, 2024

Copy link
Copy Markdown
Contributor

This node can be added after a stroke node in a set of vector data. Upon doing so, it will account for the stroke's shape and add it to the vector data. The stroke will be treated as a solid part of the shape or line segment. Both closed shapes and line segments are supported.

Closes #1092

@Keavon

Keavon commented Mar 5, 2024

Copy link
Copy Markdown
Member

Welcome, thanks for the contribution!

It looks like this does not currently compile.

@Keavon Keavon changed the title Added a Fill From Stroke Node New node: Fill From Stroke Mar 5, 2024
@Kakapio

Kakapio commented Mar 6, 2024

Copy link
Copy Markdown
Contributor Author

Welcome, thanks for the contribution!

It looks like this does not currently compile.

Sorry, had a missing import. Should compile now.

@Keavon

Keavon commented Mar 6, 2024

Copy link
Copy Markdown
Member

!build

@github-actions

github-actions Bot commented Mar 6, 2024

Copy link
Copy Markdown
📦 Build Complete for 7acb608
https://4c3f345c.graphite.pages.dev

@Kakapio

Kakapio commented Mar 6, 2024

Copy link
Copy Markdown
Contributor Author

I think I have noticed an issue where even for a closed vector shape the node assumes it's an open shape and uses the wrong vectordata....looking into this. One of my local files didn't have this issue, but new ones do. Strange.

@Kakapio

Kakapio commented Mar 6, 2024

Copy link
Copy Markdown
Contributor Author

resolved an issue. fully functional now. please rebuild.

@otdavies

otdavies commented Mar 6, 2024

Copy link
Copy Markdown
Member

!build

@github-actions

github-actions Bot commented Mar 6, 2024

Copy link
Copy Markdown
📦 Build Complete for 1a74ff6
https://3457cc2d.graphite.pages.dev

@Keavon Keavon force-pushed the fill-from-stroke branch 2 times, most recently from b074720 to 52e7224 Compare March 7, 2024 09:14
@Keavon

Keavon commented Mar 8, 2024

Copy link
Copy Markdown
Member

!build

@github-actions

github-actions Bot commented Mar 8, 2024

Copy link
Copy Markdown
📦 Build Complete for 52e7224
https://b573b226.graphite.pages.dev

@Keavon

Keavon commented Mar 8, 2024

Copy link
Copy Markdown
Member

Can you please rename this to "Solidify Stroke"?

Also, the stroke width doesn't seem to be the same. Is it perhaps being doubled (diameter being treated as radius)?

@Keavon Keavon marked this pull request as draft March 8, 2024 23:00
@Kakapio Kakapio force-pushed the fill-from-stroke branch from 52e7224 to ebcfcdd Compare March 8, 2024 23:49
@Kakapio Kakapio changed the title New node: Fill From Stroke New node: Solidify Stroke Mar 8, 2024
@Kakapio Kakapio closed this Mar 8, 2024
@Kakapio Kakapio deleted the fill-from-stroke branch March 8, 2024 23:57
@Kakapio Kakapio restored the fill-from-stroke branch March 9, 2024 00:06
@Kakapio Kakapio reopened this Mar 9, 2024
@Kakapio Kakapio force-pushed the fill-from-stroke branch from 0e950ec to 17ba1ba Compare March 9, 2024 12:40
@otdavies

otdavies commented Mar 9, 2024

Copy link
Copy Markdown
Member

!build

@github-actions

github-actions Bot commented Mar 9, 2024

Copy link
Copy Markdown
📦 Build Complete for 17ba1ba
https://2e18b657.graphite.pages.dev

@Kakapio Kakapio marked this pull request as ready for review March 11, 2024 18:07
@Keavon Keavon force-pushed the fill-from-stroke branch from 61e67c4 to 7eac88e Compare March 11, 2024 23:31
@Keavon

Keavon commented Mar 11, 2024

Copy link
Copy Markdown
Member

!build

@github-actions

Copy link
Copy Markdown
📦 Build Complete for 7eac88e
https://069a2d94.graphite.pages.dev

@Keavon

Keavon commented Mar 12, 2024

Copy link
Copy Markdown
Member

There's one bug I found, but I'm not sure if it's caused by your code: a layer with a shape and a non-1x1 transform, when you have it selected and switch to the Path tool, changes in size to gain a 1x1 transform. I'm probably just going to merge this anyways and we can work on a followup fix.

@0HyperCube do you know if that's a general bug or if that's because this node is doing (or not doing) something special?

@Keavon Keavon merged commit 2263b0a into GraphiteEditor:master Mar 12, 2024
@Kakapio Kakapio deleted the fill-from-stroke branch March 12, 2024 05:11
@Keavon

Keavon commented Jun 13, 2026

Copy link
Copy Markdown
Member

Hi @Kakapio, thanks again for this code contribution to the project! We're still hoping you will respond to the request to relicense this code. Please see #4208 ASAP, thank you!

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.

Solidify stroke node

3 participants