Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

Editor Plugin Examples using EGUI #4

Merged
merged 1 commit into from
Aug 28, 2021
Merged

Conversation

jacobsky
Copy link
Collaborator

Added two Editor Plugin samples by adding some tool scripts based on the Godot Tutorials.

This includes

  • Main Screen sample
  • Dock sample

To make this work, the GodotEguiExample and GodotEgui NativeClasses needed to be registered as tool classes.

As discussed in Discord, converting the classes to tool scripts may have some issues related to hot reloading. Based on my testing, this only appears to affect godot if you are currently using the editor actively during a build while where the native library may be deleted.

I believe this can be mitigated in real projects by either

  1. Developing toolscripts in a separate library so that they don't have to be rebuilt with your game.
  2. Avoiding pointing directly where the libraries are built and instead copy the library into the project folder (manually or via an editor build task)

Let me know if you would like to include any changes. For the plugin .cfgs, I just left the author as myself, but if you'd like to change it, I don't have a problem with it :)

@jacobsky jacobsky mentioned this pull request Aug 27, 2021
@setzer22
Copy link
Owner

Very nice! Thanks for working on this 😄

I've tested the changes and everything seems to work well. If you could you take care of the minor comments I left we should be ready to merge 👍

… the example.

- Main Screen
- Dock

To make this work, the `GodotEguiExample` and `GodotEgui` NativeClasses needed to be registered as tool classes.
@jacobsky
Copy link
Collaborator Author

@setzer22 Changes were made as requested in bd15df2

@setzer22 setzer22 merged commit 34c7772 into setzer22:main Aug 28, 2021
@setzer22
Copy link
Owner

Looks good to me!

@jacobsky jacobsky deleted the tool-sample branch August 29, 2021 12:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants