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

Add a build FGD button #55

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

krazyjakee
Copy link
Contributor

This PR adds a button to the tbloader toolbar when the tbloader node is selected addressing this issue: #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

Godot_v4.0-beta9_win64_sPtXL9R6Y1.mp4

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

@codecat
Copy link
Owner

codecat commented Dec 27, 2022

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 😜

@krazyjakee
Copy link
Contributor Author

Agreed. Made the necessary changes.

@krazyjakee
Copy link
Contributor Author

krazyjakee commented Feb 7, 2023

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.

@codecat
Copy link
Owner

codecat commented Feb 7, 2023

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

@krazyjakee
Copy link
Contributor Author

@codecat
Copy link
Owner

codecat commented Feb 7, 2023

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.

@krazyjakee
Copy link
Contributor Author

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.

@codecat
Copy link
Owner

codecat commented Feb 8, 2023

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

@HeadClot
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants