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

Fix engine_type_colors's description #98966

Merged
merged 1 commit into from
Nov 10, 2024
Merged

Conversation

Dr-helicopter
Copy link
Contributor

in editor>editor settings>Text Editor>Theme, if you hover hover over "Engine Type Color", it has "Vector2", "Vector3"and "Color" as example but these are not Engine types. and their color is controlled by the "Base Type Color" setting. I discovered this while trying to customize my editor a bit more.

@Dr-helicopter Dr-helicopter requested a review from a team as a code owner November 8, 2024 14:31
@AThousandShips AThousandShips changed the title fixed engine_type_colors's description Fix engine_type_colors's description Nov 8, 2024
@AThousandShips
Copy link
Member

Please restore the translation files, they are handled elsewhere, see the instructions

@AThousandShips AThousandShips added bug documentation cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release labels Nov 8, 2024
@AThousandShips AThousandShips added this to the 4.4 milestone Nov 8, 2024
@Dr-helicopter Dr-helicopter reopened this Nov 8, 2024
@Dr-helicopter
Copy link
Contributor Author

I have restored the translation files to their previous state(and accidentally closed my pr. I apologize for that. its my first pr)

Copy link
Member

@AThousandShips AThousandShips left a comment

Choose a reason for hiding this comment

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

Checked the code and this looks correct, just one small note

doc/classes/EditorSettings.xml Outdated Show resolved Hide resolved
@Dr-helicopter
Copy link
Contributor Author

yeah, that describes it better

@AThousandShips
Copy link
Member

Please squash your commits into one, see the interactive rebase for instructions

@Dr-helicopter Dr-helicopter force-pushed the master branch 2 times, most recently from 6c749b9 to 42ea239 Compare November 9, 2024 12:38
in editor>editor settings>Text Editor>Theme, if you hover hover over "Engine Type Color", it has "Vector2", "Vector3"and "Color" as example but these are not Engine types. and their color is controlled by the "Base Type Color" setting. I discovered this while trying to customize my editor a bit more.
@Dr-helicopter
Copy link
Contributor Author

anything else?

@AThousandShips
Copy link
Member

AThousandShips commented Nov 9, 2024

No this should be ready to go, the production team will take a look when they do a pass over PRs to merge and comment if something else is needed

@Repiteo Repiteo merged commit e2a7d51 into godotengine:master Nov 10, 2024
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 10, 2024

Thanks! Congratulations on your first contribution! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release cherrypick:4.3 Considered for cherry-picking into a future 4.3.x release documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants