Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

feat: Shortcut keys for Next/Previous Visited Frame #904

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

funatsufumiya
Copy link

@funatsufumiya funatsufumiya commented Sep 30, 2019

Hello. I added new shortcut keys for Next/Previous Visited Frame (Issue #903)

  • A or ArrowLeft - Previous Frame
  • D or ArrowRight - Next Frame
  • Q - Previous Tagged Frame
  • E - Next Tagged Frame
  • H - Previous Visited Frame (<- Added)
  • L - Next Visited Frame (<- Added)

@msftclas
Copy link

msftclas commented Sep 30, 2019

CLA assistant check
All CLA requirements met.

@wbreza
Copy link
Contributor

wbreza commented Oct 4, 2019

@funatsufumiya - Thanks for this feature. Can you please add some unit tests? There should be some similar tests you can reference for the other prev/next buttons.

@funatsufumiya
Copy link
Author

@wbreza Thank you. I'll try to fix this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants