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

Changed workflow to also work on the new guildpoint extention #15

Closed
wants to merge 37 commits into from

Conversation

klingbolt
Copy link
Owner

@klingbolt klingbolt commented Sep 25, 2024

This PR is reliant on AsherGlick#358

AsherGlick and others added 30 commits February 26, 2023 15:48
This information may be usefull elsewhere.
The value is checked to be inside the expected range.
To avoid code duplication the core functionality of _on_Dialog_hide was moved to an independet function.
This check does not hold a real value while most components of burrito only work reliable on fullscreen anyway.
Instead of calculating the position it is now directly read from a dictionary as an absolute value.
Dynamic icon positon depending on ui-scale
Dynamic minimap scale depending on ui-scale
Quick fix for nested types and bad trl paths
Update XML parsing to ignore XML prefixes from tags
Cleaning up burrito link code to make it more readable
Contains the base struct to identify the dll as a compatible addon.
@AsherGlick
Copy link

Weren't we saving .guildpoint for the full marker pack that includes the images, and not the individual binary proto files?

@klingbolt
Copy link
Owner Author

I believe that we wanted to use guildpoint to refer to the data format specifically. We had discussed .bin .data .waypoint .waypointer .somethingelse for the protobuf filename extensions and settled on .bin because we were certain on using waypoint at the time. You said during that discussion .burrito is a no because we want to save that for the .zip file repackage name (just like how .taco works) .

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.

4 participants