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

Dialogxml fixes/notes/updates #575

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

Conversation

NQNStudios
Copy link
Collaborator

@NQNStudios NQNStudios commented Feb 6, 2025

We have 4 help windows in the scenario editor that are very outdated (including #495) -- I added a cautionary dialog that warns the user the info is outdated and has the URL for openboe.com. (This is temporary and these will eventually probably call launchDocs instead of opening a help dialog at all.)

I fixed up the Hints dialog in the main game so it's up-to-date.

I removed an out-of-date Exile 3 reference from help-inventory and a TODO comment that's not relevant anymore because we do have a JOBS button.

Went through and exhaustively added a note about which dialogxmls should eventually get relative positioning if we ever need to change them. This marks the work that would be required for #300, which is not necessarily worth actually doing for the many many dialogs that just might stay the same forever.

@CelticMinstrel
Copy link
Member

Saying it here so I don't forget: I have a few changes to specific dialogs pending, which I'd like to get in before merging this.

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.

2 participants