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

Reactor can be controlled with movement keys #1533

Closed
wants to merge 3 commits into from

Conversation

JJRcop
Copy link
Contributor

@JJRcop JJRcop commented Jun 7, 2019

This happens when the mouse is near the control bars or is currently dragging one of them.

Left and Right (default A and D) control the Fission rate
Up and Down (default W and S) control the Turbine output

Run (Default LeftShift) quadruples the speed.

This is where the mouse zone is, similar to the steering menu.
image

I put the scrollbars into a GUILayoutFrame so their position was slightly shifted, I tried to get it close enough. Notably the labels are affected.

OLD
image

NEW
image

@TheHellFireo
Copy link

TheHellFireo commented Jun 8, 2019

should have PR'ed it into the dev branch

@JJRcop JJRcop changed the base branch from master to dev June 8, 2019 15:10
@JJRcop
Copy link
Contributor Author

JJRcop commented Jun 15, 2019

Changed base to dev branch and now controls still work if you're dragging one of the bars even if you move the mouse out of the detection box.

@Regalis11
Copy link
Collaborator

Nice, this looks very useful. :) We haven't had time to test and merge it yet because we have our hands full with more critical fixes atm, but we'll take a look once the more high-priority stuff is out of the way.

@JJRcop JJRcop changed the base branch from dev to master July 12, 2019 01:02
@Regalis11
Copy link
Collaborator

Merged in https://github.com/Regalis11/Barotrauma-development/commit/bb5b807aa97559e4de3b3670e038ab4d1a301db8 (private repo)

@Regalis11 Regalis11 closed this Jul 22, 2019
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.

None yet

3 participants