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

Unified pattern naming #6346

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kisslorand
Copy link
Contributor

Description

This PR unifies the pattern naming used in the support settings to be aligned with the naming and style of the infill pattern ones.

Screenshots/Recordings/Graphs


Before:

Support Base Pattern

After

Support Base Pattern

Before:

Support Interface Pattern

After:

Support Interface Pattern

Tests

Compiled and tested in Win11.


Notes

With this PR "Rectilinear Grid" and "Rectilinear Interlaced" becomes obsolete as they were supposed to do the same as the regular "Rectilinear" used in sparse infill. "Aligned Rectilinear" used in sparse infill is added to the support pattern list as this style was already present in the pattern style but under the wrong name of "Rectilinear".

@kisslorand kisslorand force-pushed the Unified-pattern-naming branch 4 times, most recently from 58c9ccc to 0145179 Compare August 4, 2024 20:59
Unified pattern naming

Unified pattern naming
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.

1 participant