C#: Move build button to EditorRunBar #79357
Conversation
raulsntos
force-pushed
the
dotnet/build-button
branch
from
0f04078 to
7dd6b4b
Compare
There was a problem hiding this comment.
Looks great to me. Makes a lot of sense.
We could consider naming the icon just Build since it's just a hammer, as it might be useful for other use cases in the future (e.g. a plugin to compile C++ GDExtension code from the editor). But that's not very important and could be changed in the future (unless we consider that EditorIcons names are part of the API and that renaming them breaks compat).
- Move C# build button to `EditorRunBar`. - Add C# build icon. - Add shortcut macros to `GodotTools`. - Move C# build shortcuts to C#.
raulsntos
force-pushed
the
dotnet/build-button
branch
from
7dd6b4b to
0359806
Compare
Keep in mind the icon is inside the .NET module so it's only included in the .NET-version of Godot. If we want the icon to be used for other cases we'll have to move it to editor. |
Right, let's not future proof then and just wait for the need to arise. |
|
Thanks! |
EditorRunBar.GodotTools.The motivation is to make the build button look more integrated with the rest of the Godot editor. Before it looked very out of place.