Skip to content

Conversation

@gucio321
Copy link
Collaborator

@gucio321 gucio321 commented Jun 25, 2025

updates to imgui v1.92.0

BREAKING

GetTexDataAsXXX has been removed according to imgui changes.

Status

  • Currently cimmarkdown doesn't compile.
  • imnodes don't compile
  • pre-compiled code should compile (?)
  • github.com/AllenDang/cimgui-go/imgui doesn't compile
  • FontBaked (function) shadows FontBaked (type). Gonna return Get prefix for function, but codegen doesn't respect replaces enough.
  • examples/common didn't compile because of textures migration, but I did a workaround. Track backend: make Texture use TextureRef instead of TextureID #449 for further changes in that topic.
  • Ebiten Backend doesn't work. I need to figure out how fonts should be handled after rework. Track Question: migrating backends to new Fonts setup ocornut/imgui#8735
    • there is no (*TextureData).Status geneated for some reason. Something wrong with gencpp. It even oesn't appear in not generated functions.
    • We needd to wrap ImVector<xxx>* to get access to ImDrawData->Textures

gucio321 and others added 11 commits June 25, 2025 21:57
updates to imgui v1.92.0
From imgui 1.92.0 release notes:
- Fonts: obsoleted ImFontAtlas::GetTexDataAsRGBA32(), GetTexDataAsAlpha8(), Build(), SetTexID() and IsBuilt() functions. The new protocol for backends to handle textures doesn't need them. Kept redirection functions (will obsolete).
  - A majority of old backends should still work with new code (behaving like they did before).
  - Calling ImFontAtlas::Build() before initializing new backends will erroneously trigger preloading all glyphs. Will be detected with an assertion after the backend is initialized.
@gucio321 gucio321 changed the title deps(update): run make update BREAKING deps(update): run make update Jun 25, 2025
@gucio321 gucio321 changed the title BREAKING deps(update): run make update **BREAKING** deps(update): run make update Jun 25, 2025
@gucio321 gucio321 changed the title **BREAKING** deps(update): run make update BREAKING deps(update): run make update Jun 25, 2025
gucio321 added 5 commits June 25, 2025 23:54
If name present in Replace table, no further changes will be done
(including removing Get prefix)
shadowing TexId field
Generally I think Backend should utilize TextureRef instead of TextureID now.
gucio321 added 4 commits June 27, 2025 16:55
- Do not call (removed) GetFontDataAsXXX
- Configure ImGui::IO by setting RendererHasTextures backend flag
this concept was entirely removed in imgui. This will be now handled through Texture Requests
@gucio321 gucio321 merged commit 2a7b09a into AllenDang:main Jun 27, 2025
5 checks passed
@gucio321 gucio321 deleted the imgui-1.92 branch June 27, 2025 22:21
gucio321 added a commit to gucio321/cimgui-go that referenced this pull request Jul 1, 2025
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.

1 participant