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

Jump to bottom

Move the definition of 'stand still' keys to where direction keys are defined #194

Open
Mikolaj opened this issue Feb 19, 2020 · 1 comment
Open

Move the definition of 'stand still' keys to where direction keys are defined #194

Mikolaj opened this issue Feb 19, 2020 · 1 comment
Labels
code smell easy good first issue help wanted self-contained UI

Comments

@Mikolaj
Copy link
Member

Mikolaj commented Feb 19, 2020

Namely, move it from

https://github.com/LambdaHack/LambdaHack/blob/v0.9.5.0/engine-src/Game/LambdaHack/Client/UI/Content/Input.hs#L59

to

https://github.com/LambdaHack/LambdaHack/blob/v0.9.5.0/engine-src/Game/LambdaHack/Client/UI/Key.hs#L211

To test, try if the M command is able to take a 'stand still' key as argument (currently only mouse permits specification of the tile under the actor).
@Mikolaj Mikolaj added this to the v0.10 milestone Aug 5, 2020
@Mikolaj Mikolaj removed the priority label Aug 5, 2020
@Mikolaj
Copy link
Member Author

Mikolaj commented Feb 10, 2021

This is too good a starter ticket to waste by implementing it, so let's remove the milestone and wait for a volunteer even after the release.

@Mikolaj Mikolaj removed this from the v0.10 milestone Feb 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
code smell easy good first issue help wanted self-contained UI
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Mikolaj