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

FF7: Add option to remove top/bottom black bars on the screen #767

Merged
merged 1 commit into from
Dec 24, 2024

Conversation

CosmosXIII
Copy link
Contributor

@CosmosXIII CosmosXIII commented Dec 21, 2024

Summary

Added uncrop option to remove black bars from top/bottom of the screen. This can be used to uncrop the screen even when using 4:3 aspect ratio or for not widescreen extended screens in 16:9 mode.

Motivation

So that people that prefer almost squared stuff can be a little more happy.

ACKs

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

@julianxhokaxhiu
Copy link
Owner

Thanks for the PR Cosmos, however is there any plan to bring this to FF8 as well? If not, I'd prefix it with ff7_ and put it in the relative category down below in the toml.

Additionally, is there any other way we can think of enabling it without an additional flag? I know they are cheap but the less they are the better is the maintainability of the code. Thank you in advance.

@julianxhokaxhiu julianxhokaxhiu added the enhancement New feature or request label Dec 21, 2024
@CosmosXIII
Copy link
Contributor Author

is there any plan to bring this to FF8 as well?

It should work in FF8 too mostly. The parts that are now FF7 only are there to fix some some backgrounds where the black bar reappear unless the camera range is corrected through the widescreen config. Now that we started FF8 widescreen implementation I think it would be better to keep the flag common or else we will end up with two separated flags for the same thing.

Additionally, is there any other way we can think of enabling it without an additional flag? I know they are cheap but the less they are the better is the maintainability of the code. Thank you in advance.

At first I thought putting it into the aspect_ratio flag but then it would get more confusing because I want it to also work when using 16:9 aspect ratio without the extended backgrounds in Limit Break.

@julianxhokaxhiu
Copy link
Owner

At first I thought putting it into the aspect_ratio flag

I was wondering why can't we just add this functionality already in the existing 4:3 check instead, as vertical bars were added only on PC if I am not mistaken, while on PSX they were never there. Since the mission here is to bring most of the PSX look and feel to PC, I think we can safely assume this mode is what everyone wants really, without adding a further flag.

Since you can also govern which FFNx flag needs to be enabled via the mod you can enable or not the 16:9 based on the case if needed. Otherwise it will just work with whichever way users might pick in the Game Driver window.

What do you think?

@CosmosXIII
Copy link
Contributor Author

CosmosXIII commented Dec 21, 2024

as vertical bars were added only on PC if I am not mistaken, while on PSX they were never there

Actually, the black bars are there too on PSX. See video below:
https://youtu.be/h0mqRkhOvtY?t=328

In fact, while most backgrounds have image data there, a few of them do not. In those cases I had to extend those to fill the top/bottom bars in Limit Break. So the idea is to activate this new flag through Limit Break mod.xml so that those few backgrounds have the extended data in the top/bottom. No need to add the flag to 7thHeaven driver settings.

@julianxhokaxhiu julianxhokaxhiu changed the title FF7: Added uncrop option to remove black bars from top/bottom of the screen FF7: Add option to remove top/bottom black bars on the screen Dec 24, 2024
@julianxhokaxhiu julianxhokaxhiu merged commit fe9c938 into julianxhokaxhiu:master Dec 24, 2024
1 check passed
@julianxhokaxhiu
Copy link
Owner

Thanks a lot :)

@julianxhokaxhiu julianxhokaxhiu added this to the 1.22.0 milestone Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants