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

Use black for font outlines by default instead of white #54641

Merged

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Nov 5, 2021

This makes font outlines more usable out of the box, as black is one of the most commonly used colors for font outlines.

Now, to configure a working font outline, you just have to increase Outline Size in a supported Control node's custom constants:

image

@KoBeWi
Copy link
Member

KoBeWi commented Feb 2, 2022

White outlines are actually useful when combined with outline modulate. Question is whether black is more common that all other colors 🤔

@Calinou
Copy link
Member Author

Calinou commented May 11, 2022

White outlines are actually useful when combined with outline modulate. Question is whether black is more common that all other colors thinking

Should we make outline modulate black by default instead (and keep the outline glyphs white)?

@YuriSizov
Copy link
Contributor

Should we make outline modulate black by default instead (and keep the outline glyphs white)?

Let's do that!

@YuriSizov YuriSizov modified the milestones: 4.0, 4.x Jan 25, 2023
@YuriSizov
Copy link
Contributor

I think this is something we can change in 4.x now.

@Mickeon
Copy link
Contributor

Mickeon commented Feb 25, 2024

A whole year has passed and it seems like this was extremely desirable back when. How do we feel about this now?

@Calinou Calinou force-pushed the gui-tweak-default-outline-color branch from 9f08bc4 to d72a03d Compare February 26, 2024 22:13
@Calinou
Copy link
Member Author

Calinou commented Feb 26, 2024

Rebased and tested again, it works as expected.

Should we make outline modulate black by default instead (and keep the outline glyphs white)?

I don't remember what this was about though. outline_modulate is only referenced in Window and Label3D (the latter is already solid black by default).

@KoBeWi
Copy link
Member

KoBeWi commented Feb 26, 2024

outline_modulate was a theme color. It was used to modulate the outline color defined in Font (which this PR modified originally I think?), but the Font outline is now gone.

@akien-mga
Copy link
Member

There are some style issues, clang-format didn't run.

Copy link
Member

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems fine to me after fixing style issues.

This makes font outlines more usable out of the box, as black
is one of the most commonly used colors for font outlines.
@Calinou Calinou force-pushed the gui-tweak-default-outline-color branch from d72a03d to 3fe0122 Compare February 28, 2024 19:25
@akien-mga akien-mga modified the milestones: 4.x, 4.3 Feb 28, 2024
@akien-mga akien-mga merged commit 05b44f3 into godotengine:master Feb 29, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

@Calinou Calinou deleted the gui-tweak-default-outline-color branch February 29, 2024 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants