-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
Input text cursor rendering issues #7031
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
Comments
We don't have a standardized mechanism to specify general/global scaling and would probably wait until we have one. Similarly I don't disagree we could have a specific color for it but it likely would have to wait until we update the style system to avoid cluttering out existing list. |
Yes but the difference is for other settings like WindowPadding, ItemSpacing... I can call ImStyle::ScaleAllSizes(). And as for fonts I can load them with scaled size. So using existing functionality I am already able to achieve DPI scaling of UI elements. But the width of the text cursor is fixed so I can't scale it unless I directly modify ImGui code. |
I would also like to have configurable cursor color. |
For scaling, I'm currently working on general scaling system and standardized scaling factor so I would like this to be part of it and not another style variable. But I could add the color sooner. |
…f the InputText cursor/caret. (#7031)
I've added a color for the cursor. |
Awesome - Thank you! |
Version/Branch of Dear ImGui:
Version: 1.90
Branch: docking
Back-end/Renderer/Compiler/OS
Back-ends: imgui_impl_android.cpp + imgui_impl_opengl3.cpp
Compiler: Android NDK Clang
Operating System: Android
My Issue/Question:
I would like the rendering of the text cursor in input controls to be more configurable.
Right now the rendering is done by drawing a 1px line. That is problematic with DPI scaling so the cursor is barely visible.
The color should be configurable too currently it is fixed to ImGuiCol_Text. Android uses different color for text cursor and text.
Screenshots/Video
The text was updated successfully, but these errors were encountered: