Bugged wrapped text layout in 1.89 #5919

New issue

Closed

opened 2022-11-25 14:21:34 +00:00 by xefinity · 1 comment

xefinity commented

2022-11-25 14:21:34 +00:00

(Migrated from github.com)

Version/Branch of Dear ImGui:

Version: 1.89.1
Branch: master

Back-end/Renderer/Compiler/OS

Back-ends: imgui_impl_opengl3.cpp (GLES2) + custom touch handling
Compiler: Clang 12 (bundled with Android NDK r23b)
Operating System: Android 10

My Issue/Question:

While testing my custom touch handling, I encountered a bug with TextWrapped rendering. If it is wrapped and scrolled past first line, then it starts rendering under other widgets from second line. It looks like to be caused by commit e0330c1, because after reverting it, I didn't encounter any issues with wrapped text rendering.

Screenshots/Video
Before reverting commit:

After reverting commit:

Standalone, minimal, complete and verifiable example:

Scroll any TextWrapped past first line in demo windows (for example, in Console example).

**Version/Branch of Dear ImGui:** Version: 1.89.1 Branch: master **Back-end/Renderer/Compiler/OS** Back-ends: imgui_impl_opengl3.cpp (GLES2) + custom touch handling Compiler: Clang 12 (bundled with Android NDK r23b) Operating System: Android 10 **My Issue/Question:** While testing my custom touch handling, I encountered a bug with TextWrapped rendering. If it is wrapped and scrolled past first line, then it starts rendering under other widgets from second line. It looks like to be caused by commit e0330c1, because after reverting it, I didn't encounter any issues with wrapped text rendering. **Screenshots/Video** _Before reverting commit:_ ![Before reverting commit](https://user-images.githubusercontent.com/75137480/204003027-eae93063-5900-4e1c-9dfd-e290702df392.png) _After reverting commit:_ ![After reverting commit](https://user-images.githubusercontent.com/75137480/204003281-4b45114b-4261-409e-93bc-a98b623b37a3.png) **Standalone, minimal, complete and verifiable example:** > Scroll any TextWrapped past first line in demo windows (for example, in Console example).

ocornut commented

2022-11-28 14:29:07 +00:00

(Migrated from github.com)

Thank you for finding this.
I have pushed the fix bd96f6e + a basic regression test in ocornut/imgui_test_engine@7eb6f2a4b5 (diff-842989679e)

Thank you for finding this. I have pushed the fix bd96f6e + a basic regression test in https://github.com/ocornut/imgui_test_engine/commit/7eb6f2a4b52e5ac5e47f61b5260c80c45892de43#diff-842989679e695d3a9befb980ce84c7a7e8575d5ad8ed4ef56bed32f6dbfb6596