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

Add autocompletion for SpriteFrames' methods #86754

Merged

Conversation

Mickeon
Copy link
Contributor

@Mickeon Mickeon commented Jan 3, 2024

Related to #86747 and #86753.

It's also why I wanted to prepare #86743.

image

I wish I could compile a build for a better showcase, but it keeps running out of memory during the compile.

This PR autocompletes the SpriteFrames methods with the names of every existing animation, if they can be detected.

I'm very fond of this. I could have really used it in the past.

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.

Also makes sense

@Mickeon
Copy link
Contributor Author

Mickeon commented Jan 3, 2024

The code style mandates all of the comparisons to be in one line, which I don't believe is ideal. I've seen some parts of the codebase do not have to do this.

@Mickeon Mickeon force-pushed the autocompletion-spriteframes-animation branch from 8b4abfe to 95fc4da Compare January 3, 2024 18:19
@Mickeon Mickeon force-pushed the autocompletion-spriteframes-animation branch from 95fc4da to c2f9a22 Compare January 3, 2024 18:25
@AThousandShips AThousandShips added this to the 4.3 milestone Jan 3, 2024
@akien-mga akien-mga merged commit 4a14f9d into godotengine:master Jan 4, 2024
15 checks passed
@akien-mga
Copy link
Member

Thanks!

@Mickeon
Copy link
Contributor Author

Mickeon commented Jan 4, 2024

If needed this & similar can be cherry-picked by the way

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