Skip to content

Conversation

RenechCDDA
Copy link
Member

Summary

None

Purpose of change

Recently I've been testing some really silly imgui stuff, which has required repeatedly setting/unsetting flags.

I copied off the imgui demo for a checkbox, but I stepped back and thought "why do I need to re-implement my checkbox every time I'm playing with one of imgui's default widgets?"

Describe the solution

Setup some generic globally-accessible/includable debug functions

Implement a cataimgui::BeginTable() alias for ImGui::BeginTable() so we can inject debug flags

add a generic set of flag checkboxes that can be inserted into imgui windows to tune behavior (gated behind debug mode so they can be safely inserted anywhere)

Move the default cataimgui::window flags to be governed by the debug settings so future imgui windows can be rapidly prototyped

(Not done) Add a simple dedicated window to access the flag settings

(Not done) Move more imgui default widgets to aliases and audit all existing calls to them

Describe alternatives you've considered

Testing

It's for debug! Testing is all I do!

Additional context

As we move to more and more windows having been ported to imgui some of these options (controlling the default window flags) may need to be removed or made more specific. But until then they should be used as much as possible

@github-actions github-actions bot added Info / User Interface Game - player communication, menus, etc. [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant