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

[WIP] Implementation of DominantHand #712

Closed

Conversation

ZephireNZ
Copy link
Contributor

Continuation of #630
SpongePowered/SpongeAPI#1121

This implements the new HandType manipulator, with potential for mods to add arbitrary hands later with Forge.

Test Plugin code:

COMING SOON TO A THEATRE NEAR YOU


@Override
public String getName() {
return this.key.replace("options.mainHand.", "");
Copy link
Contributor

Choose a reason for hiding this comment

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

Should probably set a field with this value

@Aaron1011 Aaron1011 added the status: wip Work in progress label May 13, 2016
Fix hand setting

Progress on new handedness
@ZephireNZ ZephireNZ closed this Nov 11, 2016
@ZephireNZ ZephireNZ deleted the feature/dominant-hand branch November 11, 2016 23:08
@ZephireNZ ZephireNZ restored the feature/dominant-hand branch November 11, 2016 23:30
@ZephireNZ ZephireNZ reopened this Nov 11, 2016
@ZephireNZ
Copy link
Contributor Author

Accidentally deleted the branch in my pruning, oops!

@parlough
Copy link
Contributor

@ZephireNZ Do you plan to update this bad boy? If not I can handle it.

@parlough parlough self-assigned this Nov 23, 2017
@ZephireNZ
Copy link
Contributor Author

Go for it, not really had the time for Sponge recently :/

@gabizou gabizou mentioned this pull request Nov 23, 2017
@ImMorpheus ImMorpheus added status: needs updating hey, origin changed, you need to update status: wip Work in progress and removed status: wip Work in progress labels Aug 30, 2018
@ImMorpheus
Copy link
Contributor

Superseded #2105

@ImMorpheus ImMorpheus closed this Sep 21, 2018
@ImMorpheus ImMorpheus added resolution: superseded This isssue/PR was superseded by another one and removed status: needs updating hey, origin changed, you need to update labels Sep 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resolution: superseded This isssue/PR was superseded by another one status: wip Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants