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 Load Function to Accept a callback without triggering a warning #7551

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Jatin24062005
Copy link

@Jatin24062005 Jatin24062005 commented Feb 15, 2025

Resolves #7541

Changes:
Allow loadFont() to accept a callback without triggering a warning by adding up the condition statement. which checks the load function arguments and the function

PR Checklist

Copy link

welcome bot commented Feb 15, 2025

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already. And be sure to add yourself to the list of contributors on the readme page!

@davepagurek
Copy link
Contributor

Hey @Jatin24062005, rather than making a special case for fonts here, are we able to update the doc comments for loadFont to more accurately reflect the different possible overloads?

@Jatin24062005
Copy link
Author

Hey @davepagurek ! I saw that @limzykenneth mentioned some contributors are already working on the documentation. Should I still proceed with updating the doc comments for loadFont(), or would it be better to coordinate with them to avoid duplicate efforts?

@davepagurek
Copy link
Contributor

If you're still up for working on it, we can just make sure @GregStanton and @perminder-17 know you're working on this one!

@Jatin24062005
Copy link
Author

@davepagurek I believe @GregStanton and @perminder-17 might be a better fit for handling the documentation updates, as they have more experience in this area. I haven’t worked much with graphics, which makes me a bit less confident in making the necessary changes.
However, if you can guide me on the key updates needed, I’d be happy to give it a try!

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

Successfully merging this pull request may close these issues.

2 participants