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

Widescreen: Avoid clipping world Y position for FIELD mode world script movements #500

Conversation

tangtang95
Copy link
Contributor

Summary

Fix #474 chocobo issue and fship_22 camera movement

ACKs

  • I have updated the Changelog.md file
  • I did test my code on FF7
  • I did test my code on FF8

@tangtang95
Copy link
Contributor Author

Wait, it seems there is another scene that has to be tested

@tangtang95
Copy link
Contributor Author

The scene seems fine.

@julianxhokaxhiu julianxhokaxhiu self-requested a review September 24, 2022 09:20
@julianxhokaxhiu julianxhokaxhiu added bug Something isn't working enhancement New feature or request labels Sep 24, 2022
@julianxhokaxhiu julianxhokaxhiu added this to the 1.14.0 milestone Sep 24, 2022
@tangtang95 tangtang95 force-pushed the bugfix/widescreen-chocobo-bug branch from 87d798e to c918255 Compare September 24, 2022 09:21
@tangtang95
Copy link
Contributor Author

Sorry, changed the commit message since it was kinda misleading

@julianxhokaxhiu julianxhokaxhiu changed the title Widescreen: Avoid clipping world view Y position for world view script movements Widescreen: Avoid clipping world Y position for FIELD mode world script movements Sep 24, 2022
@tangtang95
Copy link
Contributor Author

There are actually issue with a scene. But not really sure if it can be fixed easily

@tangtang95
Copy link
Contributor Author

I think it can be merged now. If there are other issues, it will be fixed next time

…pt movements

Fix julianxhokaxhiu#474 chocobo issue and fship_22 world/background movement.
@tangtang95 tangtang95 force-pushed the bugfix/widescreen-chocobo-bug branch from 91e5c52 to f5f118d Compare September 24, 2022 10:51
@tangtang95
Copy link
Contributor Author

Sorry, had a misunderstanding with what Cosmos wrote, now, the solution should be better.

@julianxhokaxhiu julianxhokaxhiu merged commit b0bec44 into julianxhokaxhiu:master Sep 24, 2022
@julianxhokaxhiu
Copy link
Owner

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ FF7 ] Collection of widescreen 16:9 bugs
2 participants