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

Expose EditorToaster interface #11051

Closed
timoschwarzer opened this issue Oct 30, 2024 · 1 comment · Fixed by godotengine/godot#98680
Closed

Expose EditorToaster interface #11051

timoschwarzer opened this issue Oct 30, 2024 · 1 comment · Fixed by godotengine/godot#98680

Comments

@timoschwarzer
Copy link

timoschwarzer commented Oct 30, 2024

Describe the project you are working on

Plugins could greatly benefit from this API to show info messages, warnings and errors as toast notifications. The API itself is already there, it just needs to be exposed to GDScript.

Example toast:
image

Describe the problem or limitation you are having in your project

Unsable to show messages in the existing toast system.

Describe the feature / enhancement and how it helps to overcome the problem or limitation

Expose the API to GDScript so plugins can use it.

Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams

N/A

If this enhancement will not be used often, can it be worked around with a few lines of script?

It's possible to build your own toast notification system but that results in bad user experience.

Is there a reason why this should be core and not an add-on in the asset library?

It's already there :slight

@Calinou
Copy link
Member

Calinou commented Oct 30, 2024

See godotengine/godot#77890, which made all push_error() and push_warning() uses in tool scripts create toasts. However, it seems a separate API would be preferred so that you can print errors/warnings without creating toasts. This would also let you create informational toasts which can't be created using push_error()/push_warning().

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

Successfully merging a pull request may close this issue.

3 participants