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

2.6 Bug Space bar does not cycle through units anymore. #9785

Closed
jsarles-code opened this issue Nov 15, 2021 · 8 comments
Closed

2.6 Bug Space bar does not cycle through units anymore. #9785

jsarles-code opened this issue Nov 15, 2021 · 8 comments
Labels
2.6 Needs Clarification Ticket is not actionable pending more information Problem A problem, bug, defect - something to fix

Comments

@jsarles-code
Copy link

Reproduction Steps

1.After launching 2.6, I started up pacific 1940 2nd edition. In 2.5 by hitting the space bar Triple A would start cycling through my available units to move, it doesn't appear to be doing this anymore. I know"." also works, but space bar was helpful in a lot of ways to speed up time spent on moves.
2.
3.

Attach a Save Game

Engine Version

Engine Version: 2.6

Is there anything else we should know?

@jsarles-code jsarles-code added the Problem A problem, bug, defect - something to fix label Nov 15, 2021
@jsarles-code
Copy link
Author

jsarles-code commented Nov 15, 2021

To clarify, you can still use the spacebar to cycle through the units but now it requires you to specifically click on a unit and then you can start skipping them. It's a subtle nuance but it's annoying.

@jsarles-code jsarles-code reopened this Nov 15, 2021
@DanVanAtta
Copy link
Member

'spacebar' was problematic because whenever a button was 'activated, spacebar would activate that button rather than doing the 'unit skip'. I think you can observe this by clicking the 'next' button and then pressing spacebar will repeat.

Use '>' and '<' to go through units.

@jsarles-code
Copy link
Author

Interesting, ok that so makes sense. I don't truly understand why that is problematic but I can understand that might have caused other issues?
I personally have been a huge fan of the spacebar, it's a much bigger area to hit on a keyboard and I think it allows me to play the game so much faster.

If we put that issue aside I have a separate issue, let me know if you would like me to create a separate issue.

so I've discovered that the space bar doesn't start skipping units until you have manually hit the skip button.
This seems like an issue right? If I start using the < and > buttons as you described , I can start cycling through units. This works well. But as soon as I as run into a unit I want to skip the user needs to physically click the on screen skip button. Once the user has clicked the "skip button" (checkmark button) on screen , only then can the user use the spacebar to skip units. This would be okay unless the user breaks this cycle.
If I repeat the steps as I've described and then I move a unit then the user loses the ability to use the spacebar to skip through units. I have to physically click on the on screen skip button and only then can I once again use the spacebar to skip.

I think this is a separate issue because even if the user cycles through their units using the < > buttons they have lost the ability to use the spacebar to skip units unless they click the screen first.
Here's the process in steps

2.5 Engine Version
Hit space bar to cycle to the next unit
Move the unit or hit space bar to skip or "s" to sleep

2.6 Engine Version
use < or > to cycle to the next units.
At this point the user must either physically move the unit or click the on screen skip button.
If the user clicks the on screen skip button (checkmark)
the user can then start using the space bar to skip the next unit that comes up

but if the user needs to move their next units, they can move them and then
cycle to the next units using the < > buttons
after this they must once again click the on screen skip button to skip their units.
The user can use the space bar repeatedly to skip through units, but if they need to move a unit that means they need to hit the onscreen button before they can once again take advantage of the space bar to skip units.

I've also created a separate issue about the sleep button missing or being deprecated? #9786

I think using the <> buttons to cycle through units to cycle makes sense.

Let me know if anything is unclear or in-precise here.
Thanks

@DanVanAtta
Copy link
Member

so I've discovered that the space bar doesn't start skipping units until you have manually hit the skip button.
This seems like an issue right? If I start using the < and > buttons as you described , I can start cycling through units. This works well. But as soon as I as run into a unit I want to skip the user needs to physically click the on screen skip button. Once the user has clicked the "skip button" (checkmark button) on screen , only then can the user use the spacebar to skip units. This would be okay unless the user breaks this cycle.

This is the problem with space bar. By default, the space bar will activate the last activated button. Then, if there is no such button, the space bar event is actually sent to our code and we can then process it as a 'skip unit event'. This why using space bar is not super ideal, but it can be unclear which button is being activated when pressing it.

I have to physically click on the on screen skip button and only then can I once again use the spacebar to skip.

Having to physically click the screen should be a problem for any of the hotkeys. I'm interested if that is not the case. This needing to click the screen is actually a major problem in multiplayer as the code in various unknown places loses screen focus, which makes hot keys feel buggy as they do not work without that magic click-on-screeen

@DanVanAtta
Copy link
Member

Given that we have removed space bar as the 'skip unit' button, does this resolve the concerns raised in this issue? What would a fix to this issue look like? I don't think it's readily possible to use the space-bar as a hotkey in a deterministic way.

@DanVanAtta DanVanAtta added the Needs Clarification Ticket is not actionable pending more information label Mar 27, 2022
@DanVanAtta DanVanAtta added this to Triage in Problem Tracker via automation Mar 27, 2022
@jsarles-code
Copy link
Author

jsarles-code commented Mar 27, 2022 via email

@DanVanAtta
Copy link
Member

Being able to skip with '>' and '<' is the same right, though with a less convenient key? Is there a bigger difference here that I'm not understanding? @jsarles-code

@jsarles-code
Copy link
Author

jsarles-code commented Mar 31, 2022 via email

@DanVanAtta DanVanAtta moved this from Triage to Back Log: Fix Sooner in Problem Tracker Jul 12, 2022
@asvitkine asvitkine added the 2.6 label Jul 4, 2023
@jsarles-code jsarles-code closed this as not planned Won't fix, can't repro, duplicate, stale Oct 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.6 Needs Clarification Ticket is not actionable pending more information Problem A problem, bug, defect - something to fix
Projects
Status: Done
Problem Tracker
Back Log: Fix Sooner
Development

No branches or pull requests

3 participants