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 the updated JRPG demo for Godot 4.0 #3

Merged
merged 3 commits into from
Oct 8, 2023
Merged

Fix the updated JRPG demo for Godot 4.0 #3

merged 3 commits into from
Oct 8, 2023

Conversation

Nonparoxysmic
Copy link

This fixes the fatal issues in the JRPG demo with updated art assets for Godot 4.

  • Bugs were fixed in GDScript files and in the scene trees.
  • Small improvements were made to some UI elements.
  • Scene files that were edited were automatically converted to Format 3 by the engine.

This work was performed and tested in Godot 4.0.4.

@Calinou
Copy link
Owner

Calinou commented Oct 4, 2023

Thanks for carrying out this work! It's highly appreciated 🙂

I've tested this branch on Godot 4.1.1 and can notice an issue with character movement. It gets softlocked if you press a movement key twice or hold it down long enough (possibly due to key echo events). The square permanently appears outside the character in this case.

simplescreenrecorder-2023-10-04_03.36.05.mp4

The NPC still animates after this softlock, but not the player sprite.

The rest (combat, finding the key) seems to work fine, although ProgressBar theming in the UI (used for health bars during combat) was lost during the transition.

@Nonparoxysmic
Copy link
Author

Thank you! This is my bad for not testing in 4.1 as well.

I think the issue is where I have the player character's movement script awaiting signals from the AnimationTree. Something appears to be different in 4.1 that is causing those signals to work in a way I didn't expect. I'll do more research and see if I can find a good solution.

I believe I can restore some theming to the health bars so I will do that also.

@Nonparoxysmic
Copy link
Author

I think I have fixed those issues now. Tested in 4.0.4 and 4.1.2.

Copy link
Owner

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

Tested on 4.1.1, works great! Thanks a lot 🙂

@Calinou Calinou merged commit 942c872 into Calinou:port-new-art-to-godot-4 Oct 8, 2023
1 check passed
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