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

Cannot use some hotkeys to choose tank to fill in a vehicle #26467

Closed
Zetsukaze opened this issue Oct 31, 2018 · 3 comments
Closed

Cannot use some hotkeys to choose tank to fill in a vehicle #26467

Zetsukaze opened this issue Oct 31, 2018 · 3 comments
Labels
<Bug> This needs to be fixed Controls / Input Keyboard, mouse, keybindings, input UI, etc. Good First Issue This is a good first issue for a new contributor (S2 - Confirmed) Bug that's been confirmed to exist Vehicles Vehicles, parts, mechanics & interactions

Comments

@Zetsukaze
Copy link
Contributor

Describe the bug
Some hotkeys can be used to select tanks, while others cannot. I have observed this on previous builds but only confirmed this for the current build.

To Reproduce
Steps to reproduce the behavior:

  1. Have a vehicle with a few tanks
  2. Examine the vehicle to bring up the vehicle screen, press [f] to refill a tank
  3. Press [g] or [h] and observe nothing happens
  4. Press [c] or [d] and observe that the screen to choose which liquid to fill appears

Expected behavior
All the hotkeys shown work

Screenshots
refill1

Versions and configuration(please complete the following information):
Windows 10, Build 8097

@Night-Pryanik Night-Pryanik added (S1 - Need confirmation) Report waiting on confirmation of reproducibility Vehicles Vehicles, parts, mechanics & interactions Controls / Input Keyboard, mouse, keybindings, input UI, etc. labels Oct 31, 2018
@kevingranade kevingranade added <Bug> This needs to be fixed Good First Issue This is a good first issue for a new contributor and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Aug 12, 2019
@kevingranade
Copy link
Member

kevingranade commented Aug 12, 2019

This is handled here

bool veh_interact::overview( std::function<bool( const vehicle_part &pt )> enable,

I'm guessing this little lambda
const auto next_hotkey = [&]( char &hotkey ) {
hotkey += 1;
if( hotkey == '{' ) {
hotkey = 'A';
}
return hotkey;
};

needs to skip over reserved keybindings.

@Night-Pryanik Night-Pryanik added the (S2 - Confirmed) Bug that's been confirmed to exist label Sep 29, 2019
@chrimchim
Copy link
Contributor

chrimchim commented Feb 20, 2020

Hello, I am an amateur coder looking to work on the project with my only experience being the free Codecademy C++ course I completed to try to learn more about it. I've been looking at this problem for a couple days and I can't see an easy way to tell which keys are bound and which aren't. I tried looking at the main inventory's code for a solution but that is in many ways more complicated and uses a lot of dependencies. I would greatly appreciate any info on where to look for solutions or where this problem is fixed in other parts of the game as I continue to figure this out.

@Ramza13
Copy link
Contributor

Ramza13 commented May 5, 2020

This looks like its been fixed but not closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Controls / Input Keyboard, mouse, keybindings, input UI, etc. Good First Issue This is a good first issue for a new contributor (S2 - Confirmed) Bug that's been confirmed to exist Vehicles Vehicles, parts, mechanics & interactions
Projects
None yet
Development

No branches or pull requests

5 participants