godot-tbloader icon indicating copy to clipboard operation
godot-tbloader copied to clipboard

Add a build FGD button

Open krazyjakee opened this issue 2 years ago • 9 comments
trafficstars

This PR adds a button to the tbloader toolbar when the tbloader node is selected addressing this issue: https://github.com/codecat/godot-tbloader/issues/4

image

Clicking this button iterates through the existing entities and creates an fgd file for use with trenchbroom. A guide to the kind of variables it can support, usage and limitation information is available here: https://github.com/krazyjakee/godot-fgd

https://user-images.githubusercontent.com/623463/209714762-f8c835f5-c42b-4886-b876-8ac31e93be1a.mp4

@codecat I know you specifically requested this functionality in C++, however, this is a working starting point.

krazyjakee avatar Dec 27 '22 19:12 krazyjakee

Thanks for the PR! It's a little weird to have this as a submodule I think, especially as the repository also includes TBLoader itself 😜

codecat avatar Dec 27 '22 22:12 codecat

Agreed. Made the necessary changes.

krazyjakee avatar Dec 28 '22 07:12 krazyjakee

The readme still reads "Since Qodot currently doesn't officially support Godot 4" which is no longer true, however, there are other valid reasons why tbloader is better than Qodot and that includes how FGD files are generated in this PR. It's zero-config and so much easier to get going without running to documentation to fill out boilerplate logic.

krazyjakee avatar Feb 07 '23 10:02 krazyjakee

Huh? Qodot supports Godot 4 now? Do you have any link for that?

codecat avatar Feb 07 '23 11:02 codecat

https://github.com/QodotPlugin/Qodot

krazyjakee avatar Feb 07 '23 11:02 krazyjakee

Oh that's awesome! I see it also requires the .Net version of Godot 4, so I suppose that's another benefit of TBLoader.

Honestly I was hoping I could eventually stop maintaining this and direct people back to Qodot 😅

We'll see how it goes. Gonna keep supporting it for some time anyway.

codecat avatar Feb 07 '23 11:02 codecat

Oh that's awesome! I see it also requires the .Net version of Godot 4, so I suppose that's another benefit of TBLoader.

Honestly I was hoping I could eventually stop maintaining this and direct people back to Qodot 😅

We'll see how it goes. Gonna keep supporting it for some time anyway.

Yes please!

I wish I had a background in C++ so I could get more involved.

krazyjakee avatar Feb 07 '23 12:02 krazyjakee

Thanks for your work on this by the way. I will do my best to review this as soon as I possibly can. 🤗

codecat avatar Feb 08 '23 14:02 codecat

Any word on when this is going to be merged into TBLoader?

HeadClot avatar Jul 31 '23 23:07 HeadClot