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

Expose functionalities of CanvasItemEditor to GDScript, through EditorInterface #68706

Closed

Conversation

xiongyaohua
Copy link
Contributor

@xiongyaohua xiongyaohua commented Nov 16, 2022

Writing 2D editor plugins in GDScript use some hacky work around to interact with CanvasItemEditor. This PR exposes CanvasItemEditor directly, This PR exposes some functionalities of CanvasItemEditor through EditorInterface, in order to make life of 2D plugin writer easier.

Currently only methods that are essential for 2D editor plugin are exposed. More functionalities (such as snapping and tool state) can be exposed if there is demand. See godotengine/godot-proposals#1302 for discussion.

Another PR address the same problem with different ways. I am not familiar with Godot enough to tell which is the better way. Hope reviewer could shed some light.

@xiongyaohua xiongyaohua requested a review from a team as a code owner November 16, 2022 01:43
@xiongyaohua xiongyaohua marked this pull request as draft November 16, 2022 01:49
@xiongyaohua xiongyaohua marked this pull request as ready for review November 16, 2022 02:14
@Calinou Calinou added this to the 4.0 milestone Nov 16, 2022
@YuriSizov YuriSizov modified the milestones: 4.0, 4.x Nov 16, 2022
@xiongyaohua xiongyaohua changed the title Expose CanvasItemEditor to GDScript Expose functionalities of CanvasItemEditor through EditorInterface to GDScript Nov 18, 2022
@xiongyaohua xiongyaohua changed the title Expose functionalities of CanvasItemEditor through EditorInterface to GDScript Expose functionalities of CanvasItemEditor to GDScript, through EditorInterface Nov 18, 2022
@xiongyaohua xiongyaohua marked this pull request as draft November 18, 2022 04:49
@xiongyaohua xiongyaohua force-pushed the expose_canvasitemeditor branch from 640e92f to b3f0cbd Compare November 18, 2022 05:30
@xiongyaohua xiongyaohua marked this pull request as ready for review November 18, 2022 05:38
@xiongyaohua xiongyaohua force-pushed the expose_canvasitemeditor branch 3 times, most recently from 079e825 to 66b11ae Compare November 23, 2022 11:45
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.

3 participants