Skip to content

Add Escape hotkey key as a workflow reset.#286

Open
tedwaine wants to merge 1 commit into
AcademySoftwareFoundation:developfrom
tedwaine:ken_escape_reset
Open

Add Escape hotkey key as a workflow reset.#286
tedwaine wants to merge 1 commit into
AcademySoftwareFoundation:developfrom
tedwaine:ken_escape_reset

Conversation

@tedwaine

Copy link
Copy Markdown
Collaborator

This adds an Escape hotkey that doesn the following:

  • stopping playback
  • exiting presentation mode if active
  • making the viewed container the inspected container
  • resetting the compare mode to the default for viewed container type
  • soloing the selection to just the single hero selected media item
  • leaving pinned-source mode if active

In a dailies environment, when I'm done reviewing a submission I found that I needed to go through quite a few steps before moving on to the next submission. And these step often involved a cumbersome back-and-forth between mouse and keyboard. Now when I want to move on I just hit Escape and the Down arrow key.

This is possibly a subjective improvement, but hopefully OK to merge since the Escape key is only currently used by the grading tool to cancel construction of a polygon mask shape. This commit includes a modification to that code so that it works when needed without doubling up.

This adds an Escape hotkey that doesn the following:
  - stopping playback
  - exiting presentation mode if active
  - making the viewed container the inspected container
  - resetting the compare mode to the default for viewed container type
  - soloing the selection to just the single hero selected media item
  - leaving pinned-source mode if active

In a dailies environment, when I'm done reviewing a submission I found that I needed to go through quite a few steps before moving on to the next submission. And these step often involved a cumbersome back-and-forth between mouse and keyboard. Now when I want to move on I just hit Escape and the Down arrow key.

This is possibly a subjective improvement, but hopefully OK to merge since the Escape key is only currently used by the grading tool to cancel construction of a polygon mask shape. This commit includes a modification to that code so that it works when needed without doubling up.

Signed-off-by: Ken McGaugh <ken@mcgaugh.co.uk>
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.

2 participants