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

Missing word in alert popup #79380

Closed
WiseNoodle opened this issue Jul 12, 2023 · 3 comments · Fixed by #79381
Closed

Missing word in alert popup #79380

WiseNoodle opened this issue Jul 12, 2023 · 3 comments · Fixed by #79381

Comments

@WiseNoodle
Copy link
Contributor

WiseNoodle commented Jul 12, 2023

Godot version

v4.1.stable.official

System information

Godot v4.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated AMD Radeon (TM) RX 480 (Advanced Micro Devices, Inc.; 31.0.14057.5006) - Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz (8 Threads)

Issue description

There is a missing word in an alert popup. Will include a screenshot.

The alert should read, "Missing file '(filename' or one of its dependencies".

Instead it reads, "Missing file '(filename' or one its dependencies".

(it's missing the word "of").

NOTE: THE IMPORT ERROR IS NOT THE ISSUE, ITS THE WORDING IN THE ALERT POPUP.

This report should be of LOW priority.

(p.s. If I'm missing any info you'll have to forgive me; this is my first issue/bug report.

image

Steps to reproduce

Using your file manager, copy a scene with a script attached (just the .tscn and .gd files) into a different project's root res:// folder.

Then try opening the pasted scene and you will get the alert.

Minimal reproduction project

N/A requires two projects.

@WiseNoodle WiseNoodle changed the title Misspelling/improper grammar in alert popup Missing word in alert popup Jul 12, 2023
@WiseNoodle
Copy link
Contributor Author

Can I fix this myself and create a pull request? I've always wanted to become a contributor

@AThousandShips
Copy link
Member

Go ahead! Make sure to follow the guidelines

@WiseNoodle
Copy link
Contributor Author

No way! Sick!! And will do. So hyped

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