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

Add keyboard controls to TestBrowser #1287

Merged
merged 4 commits into from Dec 25, 2017

Conversation

2 participants
@peppy
Member

peppy commented Dec 24, 2017

  • Ctrl-F to search (and hit enter to load first match)
  • Ctrl-H to hide test listing
  • Ctrl-R to re-run current test
localQueue.Reverse();

This comment has been minimized.

@smoogipoo

smoogipoo Dec 25, 2017

Contributor

Are you sure about these changes?

@smoogipoo

smoogipoo Dec 25, 2017

Contributor

Are you sure about these changes?

This comment has been minimized.

@peppy

peppy Dec 25, 2017

Member

These were left over from when this was an InputManager, not Container (and adding self would break things). The change is made to allow a KeyBindingContainer that can also handle bindings itself, avoiding the need for two layers of container to do this.

@peppy

peppy Dec 25, 2017

Member

These were left over from when this was an InputManager, not Container (and adding self would break things). The change is made to allow a KeyBindingContainer that can also handle bindings itself, avoiding the need for two layers of container to do this.

@smoogipoo smoogipoo merged commit e5439cf into ppy:master Dec 25, 2017

1 check passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@peppy peppy deleted the peppy:test-browser-keyboard-control branch Jan 11, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment