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

[Merged by Bors] - Alien cake addict: Allow holding movement keys #2072

Closed

Conversation

RedlineTriad
Copy link
Contributor

I wanted to try one of the new examples but it felt so clunky that I wanted to improve it.

It did make me feel like maybe some input axes abstraction like Unity has might be useful.

Also, eating cake should probably be a separate system from movement.

@alice-i-cecile alice-i-cecile added the C-Examples An addition or correction to our examples label May 1, 2021
@mockersf
Copy link
Member

mockersf commented May 1, 2021

please no WASD 😞

@RedlineTriad
Copy link
Contributor Author

please no WASD disappointed

Why?

@mockersf
Copy link
Member

mockersf commented May 1, 2021

Because it doesn't make any sense on anything but QWERTY.

@RedlineTriad
Copy link
Contributor Author

Also works for QWERTZ, but until we have a position based keyboard input system this is probably the best we can do.

@mockersf
Copy link
Member

mockersf commented May 1, 2021

because it's not the best we can do, supporting wasd makes a lot of games unplayable without changing layout and it's not something I would like to promote in the examples. best we can do is arrows

@RedlineTriad
Copy link
Contributor Author

because it's not the best we can do, supporting wasd makes a lot of games unplayable without changing layout and it's not something I would like to promote in the examples. best we can do is arrows

I feel like it will be hard to do until Bevy has something like https://docs.unity3d.com/Packages/com.unity.inputsystem@1.1/manual/Keyboard.html

The location of individual keys is agnostic to keyboard layout. This means that, for example, the A key is always the key to the right of the Caps Lock key, regardless of where the currently active keyboard layout places the key that generates an a character, or whether or not the layout doesn't have a key assigned to that character.

But for me WASD is also much more comfortable because it's where I normally place my left hand.

Also is this meant as PR feedback or just as a discussion?

@mockersf
Copy link
Member

mockersf commented May 1, 2021

Also is this meant as PR feedback or just as a discussion?

Both: I would really like to not have this in the example, but I can understand it makes sense for part of the world. I also hold no special power to stop it, so it's just my preference. And it's not on a technical point, so I won't put it as a review asking for change.

Just a very strong opinion and preference on my side

@RedlineTriad
Copy link
Contributor Author

RedlineTriad commented May 1, 2021

I feel like it's worth it since it makes the game nicer for people with QWERTY/QWERTZ and doesn't really hurt people who have a different layout since arrows are still available.

And I don't feel bad about having this code in the examples (where it might be copied from) because we can't really tell developers not to use WASD for movement because Bevy doesn't have an alternative I know of and for movement it's the de-facto standard.

@Dispersia
Copy link
Contributor

Dispersia commented May 2, 2021

I was interested so googled it, and it appears the french keyboard is the only standardized keyboard that isn't qwerty/qwertz.

Regardless of that fact, my 60% keyboard doesn't have arrow keys, so that isn't standardized either. If those keys aren't being used for anything else in the demonstration, and the fact that 98% of keyboards would match the schema, I, personally at least, see no harm in it being added especially since it's not like the arrow keys are no longer used.

@mockersf
Copy link
Member

mockersf commented May 2, 2021

the french keyboard is the only standardized keyboard that isn't qwerty/qwertz.

Then it makes sense, I'll go back to my French corner with my French keyboard...

@cart
Copy link
Member

cart commented May 2, 2021

@mockersf your "french keyboard perspective" would be valuable in this conversation: #2052.

I'm particularly curious about the physical positions of "scan codes" on french keyboards.

WASD KeyCodes don't just alienate french people. They also alienate "alternative keyboard layout users" like dvorak and colemak. I think we should resolve the "physical key position" conversation in #2052 before making a call on WASD KeyCodes here. Using a pattern in an example is akin to endorsing that pattern and we should encourage people to do the "right thing" here (whatever that ends up being).

@cart
Copy link
Member

cart commented May 2, 2021

(of course supporting re-mapping is a critical part of the "final" solution)

@cart cart added the S-Pre-Relicense This PR was made before Bevy added the Apache license. Cannot be merged or used for other work label Jul 23, 2021
@mockersf mockersf removed the S-Pre-Relicense This PR was made before Bevy added the Apache license. Cannot be merged or used for other work label Jul 24, 2021
@RedlineTriad
Copy link
Contributor Author

RedlineTriad commented Jan 13, 2022

@mockersf Should I just remove WASD? I don't see any progress being made on #2052 so I think it's better to close this PR one way or another.

The WASD keys are convenient for QWERTY keyboards,
but this might not be the case for other layouts.

Until there is a way to use key positions instead of letters
we will not showcase it in examples.
@RedlineTriad RedlineTriad changed the title Alien cake addict: Allow holding movement keys and add WASD movement Alien cake addict: Allow holding movement keys Feb 15, 2022
@alice-i-cecile
Copy link
Member

bors try

bors bot added a commit that referenced this pull request Feb 15, 2022
@alice-i-cecile
Copy link
Member

Tested locally; this feels much better and is now just uncontroversial QoL and code quality fixes.

@alice-i-cecile
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Feb 15, 2022
I wanted to try one of the new examples but it felt so clunky that I wanted to improve it.

It did make me feel like maybe some input axes abstraction like Unity has might be useful.

Also, eating cake should probably be a separate system from movement.
@bors bors bot changed the title Alien cake addict: Allow holding movement keys [Merged by Bors] - Alien cake addict: Allow holding movement keys Feb 15, 2022
@bors bors bot closed this Feb 15, 2022
@RedlineTriad RedlineTriad deleted the alien-cake-addict-movement branch February 16, 2022 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Examples An addition or correction to our examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants