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

Disable polygon editing on foreign resources #63252

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

Conversation

SaracenOne
Copy link
Member

Disables polygon edits on foreign resources since the results will not be saved.

@SaracenOne SaracenOne force-pushed the disable_foreign_polygon_edits branch from 2a852a4 to 56c65d9 Compare July 21, 2022 14:56
@KoBeWi
Copy link
Member

KoBeWi commented Aug 2, 2022

Seeing the duplicate code here, I think there should be some method for checking whether a resource is foreign (either in Resource or EditorNode, or InspectorDock?). Something like is_foreign(resource, for_scene) where for_scene would usually be the current scene.

@reduz
Copy link
Member

reduz commented Aug 8, 2022

Polygon editing should work as long as you re-create the polygon resource. If the resource was created locally, it will be saved without an issue.

@reduz
Copy link
Member

reduz commented Aug 8, 2022

I think the trend should somehow be the same with this and the previous PR. If the subresource exists, wipe it before editing it and it should work. Revert icon should work fine too to get the original back.

@YuriSizov YuriSizov modified the milestones: 4.0, 4.1 Feb 9, 2023
@YuriSizov YuriSizov modified the milestones: 4.1, 4.2 Jun 14, 2023
@AThousandShips AThousandShips modified the milestones: 4.2, 4.3 Oct 26, 2023
@KoBeWi KoBeWi modified the milestones: 4.3, 4.x Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants