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

Control preset possibility and shield powerup implementation #145

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

hooger
Copy link

@hooger hooger commented Feb 12, 2019

I created the control preset option (possibility to use multiple kay mapping) to make it possibly to have a controller preset as well. That is a bit stalled...

I also created a shield power up, with icon and some animation.

Gergely Horváth and others added 9 commits May 8, 2018 18:20
modified InputHandler::Binding, to handle preset storage
modified every code using InputHandler::Binding
modified InputHandler::getKeyName function and every usage
modified Configuration::ReadPlayerBinding to read multiple presets
modified Configuration::ReadPlayerBinding to write multiple presets
modified InputHandler::processEvent to check every preset for match
created macro to define number of presets
removed code regarding hardwired number of presets (TODO except the switch statement for preset increment)
set number of presets to 2
changed hud msg accordingly
@ooxi
Copy link
Owner

ooxi commented Feb 17, 2019

Thank you very much for your huge engagement! Let us split this pull request into topic specific pull requests. For example a3b1bc changes lines, added by 1be91f.

How about you do a clean pull request containing only the control preset and then a pull request containing only the shield powerup?

Horvath Gergely and others added 4 commits February 24, 2019 20:36
modifications are made to store preset configurations
modified InputHandler::Binding, to handle preset storage
modified every code using InputHandler::Binding
modified InputHandler::getKeyName function and every usage
modified Configuration::ReadPlayerBinding to read multiple presets
modified Configuration::ReadPlayerBinding to write multiple presets
modified InputHandler::processEvent to check every preset for match
created macro to define number of presets
removed code regarding hardwired number of presets (TODO except the switch statement for preset increment)
set number of presets to 2
@hooger
Copy link
Author

hooger commented Feb 24, 2019

Hello,

sry for the late reply. I reordered the commits and moved some unnecessarry ones to a different branch. Now how do I make a pull request from earlier commits? If you could help with this I would gladly create the 2 pull requests for the distinct features.

Thanks in advance,
hooger

@hooger
Copy link
Author

hooger commented Mar 4, 2019

I created 2 separate PRs, tell me if I need to make some more changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants