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

Expand top-down controls. #267

Merged
merged 2 commits into from
May 31, 2016
Merged

Expand top-down controls. #267

merged 2 commits into from
May 31, 2016

Conversation

oznogon
Copy link
Contributor

@oznogon oznogon commented May 31, 2016

  • Allow for an unlocked camera.
  • Add keyboard controls for panning and zooming.
  • Add mouse wheel zoom controls.
  • Add a selector for picking a player ship to lock onto.

@oznogon oznogon force-pushed the top-down-ui branch 2 times, most recently from 513cdd1 to 2a24525 Compare May 31, 2016 09:51
-   Allow for an unlocked camera.
    -   Add keyboard controls for panning and zooming.
    -   Add mouse wheel zoom controls.
-   Add a selector for picking a player ship to lock onto.
@daid
Copy link
Owner

daid commented May 31, 2016

The idea of the topdown view is actually to run it on a different monitor/tv where people can follow the action "outside" of the main game. So I'm not sure how happy I am with controls on top of this view.

I did want to expand it so it zooms/pans automatically to better follow action happening.

-   Hide GUI controls for selecting player ships and locking the camera.
-   Toggle GUI visibility with the H key.
-   Cycle back and forth through the list of player ships with the J and K keys.
-   Toggle camera locking with the L key.
@oznogon
Copy link
Contributor Author

oznogon commented May 31, 2016

Thanks for the feedback, @daid. I've added keyboard controls to toggle the camera lock and cycle through player ships, and I've hidden the GUI elements by default. They can still be revealed with the H key if a spectator wants them.

@daid daid merged commit 10bd391 into daid:master May 31, 2016
@daid
Copy link
Owner

daid commented May 31, 2016

Sounds good :)

@oznogon oznogon deleted the top-down-ui branch June 13, 2016 22:48
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