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

FF8: vibration option in main menu #658

Conversation

myst6re
Copy link
Contributor

@myst6re myst6re commented Feb 18, 2024

Summary

Bring back Vibration option in the main menu and the battle pause menu.

Motivation

The vibration option was restored only in field and worldmap pause menus. Since it is common that user want to disable vibration in battle, it is great to restore the original pause menu in battle too.

ACKs

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

Images

ePSXe french
ePSXe french
FFNx french enabled
FFNx french enabled
FFNx french disabled (keyboard or gamepad without rumble capability)
FFNx french disabled

@myst6re myst6re force-pushed the feature/ff8-vibration-option-in-main-menu branch 4 times, most recently from 75c8439 to cb92f4c Compare February 21, 2024 23:17
@myst6re
Copy link
Contributor Author

myst6re commented Feb 22, 2024

If no vibrate capability
2024-02-22 21_33_49-Final Fantasy VIII
If gamepad with vibrate capability
2024-02-20 22_48_56-Final Fantasy VIII
ePSXe
2024-02-20 20_40_59-ePSXe - Enhanced PSX emulator

@myst6re myst6re marked this pull request as ready for review February 22, 2024 20:39
@myst6re myst6re force-pushed the feature/ff8-vibration-option-in-main-menu branch 2 times, most recently from c6ac4d9 to 76bab11 Compare February 23, 2024 18:29
@julianxhokaxhiu julianxhokaxhiu added the enhancement New feature or request label Feb 25, 2024
@julianxhokaxhiu julianxhokaxhiu added this to the 1.19.0 milestone Feb 25, 2024
@julianxhokaxhiu
Copy link
Owner

Can you please rebase and update the changelog? Thanks!

@myst6re myst6re force-pushed the feature/ff8-vibration-option-in-main-menu branch from 76bab11 to d7d1ac5 Compare February 25, 2024 16:51
@myst6re myst6re force-pushed the feature/ff8-vibration-option-in-main-menu branch from d7d1ac5 to 5ea2924 Compare February 25, 2024 16:52
@myst6re
Copy link
Contributor Author

myst6re commented Feb 25, 2024

Can you please rebase and update the changelog? Thanks!

Done!

@julianxhokaxhiu julianxhokaxhiu merged commit 55fcff7 into julianxhokaxhiu:master Feb 25, 2024
@julianxhokaxhiu
Copy link
Owner

Thank you!

@myst6re myst6re deleted the feature/ff8-vibration-option-in-main-menu branch March 14, 2024 15:36
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.

None yet

2 participants