Skip to content

Adds a Toggle for Inner Radius in Polygon tool (#3484)#3491

Open
Thunder-Blaze wants to merge 1 commit intoGraphiteEditor:masterfrom
Thunder-Blaze:add-inner-radius-toggle
Open

Adds a Toggle for Inner Radius in Polygon tool (#3484)#3491
Thunder-Blaze wants to merge 1 commit intoGraphiteEditor:masterfrom
Thunder-Blaze:add-inner-radius-toggle

Conversation

@Thunder-Blaze
Copy link
Copy Markdown

Description

  • Adds a Toggle to switch between inner and outer radius in a polygon

Issue

Closes #3484

Notes For Maintainer

  • Umm, I wasn't sure if it was better to overwrite the default radius with the improvised one or create a new var. For now, at most places I created a variable as effective_radius, let me know if I go with overwriting approach
  • Also, the toggle could use a better name...

Preview

recording_2025-12-17_19.32.26.mp4

@Thunder-Blaze
Copy link
Copy Markdown
Author

Umm hey @Keavon
Can you review this PR and #3482 ?
Is there something I need to fix or any reason for which it still haven't gotten reviewed ?

@Thunder-Blaze Thunder-Blaze force-pushed the add-inner-radius-toggle branch 2 times, most recently from 2f97de1 to dd87323 Compare February 20, 2026 21:40
@Keavon Keavon force-pushed the master branch 6 times, most recently from d6228da to e58c1de Compare March 16, 2026 23:03
@Keavon Keavon force-pushed the master branch 5 times, most recently from 9b97ab7 to 2e842cb Compare March 19, 2026 11:00
@Thunder-Blaze Thunder-Blaze force-pushed the add-inner-radius-toggle branch from dd87323 to 7aa0476 Compare March 30, 2026 21:57
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.

Polygon: Use inner instead of outer radius

1 participant