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.

Sign up for GitHub

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

Jump to bottom

Improve Layer Merging Reusability & Adjust Segment Insertion Logic #2245

Open

Sidharth-Singh10 wants to merge 2 commits into GraphiteEditor:master from Sidharth-Singh10:merge_layers

+107 −132

Contributor

Sidharth-Singh10 commented Feb 1, 2025

Addresses: GraphiteEditor/Graphite#2227 (comment)

Changes:

Moved the merge_layers function from pen_tool to common_functionality/graph_modification_utils to improve reusability.
Updated logic to merge layers before inserting a segment when connecting endpoints from different layers.

Sidharth-Singh10 added 2 commits

February 1, 2025 13:23


          move merge_layers function to graph_modification_utils

242d315


          merge_layer before segment insertion

91da3a6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet