FreeIntv icon indicating copy to clipboard operation
FreeIntv copied to clipboard
verified publisher icon libretro

[WIP DRAFT] Add game specific mappings to suit gamepads, Add XBox style controll layout, Upped version

Open BlazeEntertainment opened this issue 4 years ago • 5 comments

  • Added Game specific mappings. In games like Night Stalker, you need to use the keypad to fire the weapon which is near impossible when using a keypad or keyboard. So wrote up specific key mappings to face buttons that make sense.
  • Added XBOX style controller prompts to in game menu. A simple ifdef that can be enabled to flip the face buttons if the device uses XBOX style face button layouts instead of RetroPad
  • Upped version to 1.2, because we added savestates and overhauled a number of elements and fixed a number of graphical, sound and accuracy issues ontop of the WIP mappings.

Todo

  • Some games won't work well with modern game controllers without a keypad. These games have been remapped to facebuttons to be functional like Night Stalker, however these fixes are currently achieved with #IFDEF's. Need to replace these IFDEFS with some form of decent game detection so it automatically sets these new mappings.

BlazeEntertainment avatar Jul 01 '21 17:07 BlazeEntertainment

The statement about controllers not using the retropad doesn't make sense. All libretro controller input is via the retropad, and Xbox controllers can be mapped to the retropad as well as any other.

On Thu, Jul 1, 2021, 1:15 PM BlazeEntertainment @.***> wrote:

  • Added Game specific mappings. In games like Night Stalker, you need to use the keypad to fire the weapon which is near impossible when using a keypad or keyboard. So wrote up specific key mappings to face buttons that make sense.
  • Added XBOX style controller prompts to in game menu. A simple ifdef that can be enabled to flip the face buttons if the device uses XBOX style face button layouts instead of RetroPad
  • Upped version to 1.2, because we added savestates and overhauled a number of elements and fixed a number of graphical, sound and accuracy issues ontop of the WIP mappings.

Todo

  • Some games won't work well with modern game controllers without a keypad. These games have been remapped to facebuttons to be functional like Night Stalker, however these fixes are currently achieved with #IFDEF's. Need to replace these IFDEFS with some form of decent game detection so it automatically sets these new mappings.

You can view, comment on, or merge this pull request online at:

https://github.com/libretro/FreeIntv/pull/60 Commit Summary

  • Fix controls for some games, bump up version, add to-do
  • Fix slapshot instructions
  • Update and fix slapshot mapping
  • Remove leftover Makefile modification

File Changes

Patch Links:

  • https://github.com/libretro/FreeIntv/pull/60.patch
  • https://github.com/libretro/FreeIntv/pull/60.diff

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/libretro/FreeIntv/pull/60, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEVGC5V4FWZ4FBHKB3MLZELTVSPCNANCNFSM47VEIKUA .

markwkidd avatar Jul 01 '21 17:07 markwkidd

Appologies @markwkidd, I worded the MR extremely badly. I messaged you in person however please ignore this commit for now as upon reflection I want to reword some of these changes to make more sense and add some of the menu layout changes as a Core Option instead of an ifdef.

BlazeEntertainment avatar Jul 02 '21 20:07 BlazeEntertainment

I will have to think some more about this. I think it'd be a fundamental mistake to start dirtying up the codebase with all sorts of ifdefs for certain controllers. Best there is some libretro API extension instead to accomodate something like that. Anyway, we have to put this on hold for now until we can internally think of some solution.

inactive123 avatar Jul 03 '21 02:07 inactive123

This pull request fixes 3 alerts when merging a18e0b1fdbb528e8caf799f966eacad5d71ca23c into 5fc8d85ee9699baaaf0c63399c364f456097fc1e - view on LGTM.com

fixed alerts:

  • 3 for Short global name

lgtm-com[bot] avatar Jul 19 '21 16:07 lgtm-com[bot]

This pull request fixes 4 alerts when merging 87fe95e0ec20516c2fab7d1aa4813382bafdb2ef into 5fc8d85ee9699baaaf0c63399c364f456097fc1e - view on LGTM.com

fixed alerts:

  • 4 for Short global name

lgtm-com[bot] avatar Jul 21 '21 13:07 lgtm-com[bot]