Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implemented analogue camera controls in battle mode #340

Merged

Conversation

CosmosXIII
Copy link
Contributor

I added analogue controls for the battle camera. You can rotate the camera with the controller right stick and zoom-in/ zoom-out with the analogue triggers. Controls are only enabled when no camera animation is being executed.

@julianxhokaxhiu julianxhokaxhiu self-requested a review January 10, 2022 10:10
@julianxhokaxhiu julianxhokaxhiu added the enhancement New feature or request label Jan 10, 2022
@julianxhokaxhiu julianxhokaxhiu added this to the 1.11 milestone Jan 10, 2022
Copy link
Owner

@julianxhokaxhiu julianxhokaxhiu left a comment

Choose a reason for hiding this comment

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

LGTM

@julianxhokaxhiu julianxhokaxhiu merged commit 24bda3e into julianxhokaxhiu:master Jan 10, 2022
@julianxhokaxhiu
Copy link
Owner

Thank you!

Copy link
Contributor

@sithlord48 sithlord48 left a comment

Choose a reason for hiding this comment

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

Awesome Feature !

@@ -248,20 +286,24 @@ struct ff7_gamepad_status* ff7_update_gamepad_status()
ff7_externals.gamepad_status->button5 = gamepad.IsPressed(XINPUT_GAMEPAD_LEFT_SHOULDER); // L1
ff7_externals.gamepad_status->button6 = gamepad.IsPressed(XINPUT_GAMEPAD_RIGHT_SHOULDER); // R1
ff7_externals.gamepad_status->button7 = gamepad.leftTrigger > 0.85f; // L2
ff7_externals.gamepad_status->button8 = gamepad.rightTrigger > 0.85f; // R2

// Remap R2 trigger to R3 so that we can use the trigger for zoom-in when analogue controls are enabled
Copy link
Contributor

Choose a reason for hiding this comment

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

Please do not change controls if a feature is active.
What happens if this is enabled and i want to remap w/ R2 / L2 do in battle ?

I suggest
Either use L3 / R3 for zoom
or
Hold R2 or L2 and use the right stick's up / down for zoom

@Tharsos83
Copy link

Hi, I noticed the following about this feature: The speedhack is activated via controller with L3 + L2/R2 (Playstation). L2/R2 buttons are also used for zooming in fights. This means that you always zoom in during battle when you switch on the speedhack.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants