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

Fetch argument options from ThemeDB #85015

Conversation

KoBeWi
Copy link
Member

@KoBeWi KoBeWi commented Nov 17, 2023

#82402 (comment)
Also adds missing autocompletion for icon methods.

@KoBeWi KoBeWi added this to the 4.3 milestone Nov 17, 2023
@KoBeWi KoBeWi requested a review from a team as a code owner November 17, 2023 13:48
@KoBeWi KoBeWi force-pushed the get_argument_options_for_your_invalid_argument branch from c581ab2 to 44395b8 Compare November 17, 2023 14:18
@YuriSizov YuriSizov self-requested a review November 18, 2023 13:18
Copy link
Contributor

@YuriSizov YuriSizov left a comment

Choose a reason for hiding this comment

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

Some changes are required, plus a rebase, but otherwise it works as expected.

Edit: Also see #84763 (review), more changes are necessary.

scene/gui/control.cpp Outdated Show resolved Hide resolved
scene/gui/control.cpp Outdated Show resolved Hide resolved
scene/gui/control.cpp Outdated Show resolved Hide resolved
scene/gui/control.cpp Outdated Show resolved Hide resolved
@KoBeWi
Copy link
Member Author

KoBeWi commented Jan 31, 2024

Reordered the calls and rebased on top of e86750d
The other PR should be merged first, so I'll put it as draft for now.

@KoBeWi KoBeWi force-pushed the get_argument_options_for_your_invalid_argument branch from 44395b8 to 642b066 Compare January 31, 2024 13:41
@KoBeWi KoBeWi marked this pull request as draft January 31, 2024 13:42
@KoBeWi KoBeWi force-pushed the get_argument_options_for_your_invalid_argument branch from 642b066 to 940329c Compare February 13, 2024 17:56
@KoBeWi KoBeWi marked this pull request as ready for review February 13, 2024 17:56
@KoBeWi KoBeWi force-pushed the get_argument_options_for_your_invalid_argument branch from 940329c to fc3049d Compare May 7, 2024 12:54
@KoBeWi KoBeWi force-pushed the get_argument_options_for_your_invalid_argument branch from fc3049d to 82b6f9d Compare May 7, 2024 13:02
@akien-mga akien-mga merged commit edc61aa into godotengine:master May 7, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

@KoBeWi KoBeWi deleted the get_argument_options_for_your_invalid_argument branch May 7, 2024 14:56
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.

3 participants