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

"Search Help" - Bad selection color #21000

Closed
AlexHolly opened this issue Aug 14, 2018 · 3 comments · Fixed by #21195
Closed

"Search Help" - Bad selection color #21000

AlexHolly opened this issue Aug 14, 2018 · 3 comments · Fixed by #21195

Comments

@AlexHolly
Copy link
Contributor

9f8e2d5

image

If someone works in a bright environment the selection color(black) is almost not visible on the greyish background. (thinking maybe I should look for a basement to solve this issue thinking )

@sporkl
Copy link

sporkl commented Aug 14, 2018

I found that replacing all instances of "selection_color" with another color in editor/editor_help.cpp worked (lines 1784, 1866, 1932, 1951). (I used "success_color")

@AlexHolly
Copy link
Contributor Author

AlexHolly commented Aug 14, 2018

It points to a specific path "text_editor/theme/selection_color".
I don't think using a different name is a good idea.
The setting in the editor is missing too so I added it.
I am not sure if this is right but it works.

https://github.com/godotengine/godot/blob/master/editor/editor_settings.cpp#L360
_initial_set("text_editor/theme/selection_color", Color::html("006400"));

Replacing all 4 lines from @ComradeSparklePony
https://github.com/godotengine/godot/blob/master/editor/editor_help.cpp#L1784
class_desc->add_color_override("selection_color", EditorSettings::get_singleton()->get("text_editor/theme/selection_color"));

What would be a good default color?

@Calinou
Copy link
Member

Calinou commented Aug 17, 2018

What would be a good default color?

You could try a semi-transparent gray color such as Color::html("40808080"), this way, it works on both dark and light backgrounds. However, it will become less visible in low-contrast themes (as the background color becomes closer to gray).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants