Hotkeys not working: Z, PageUp, PageDown, =

Bug Report

Issue 1 - Description:

Hotkeys not working on the Viewer page: Z, PageUp, PageDown.

Steps to Reproduce

1. Open a study
2. Try to use the following hotkeys: Z, PageUp, PageDown
3. Checks that nothing happened.

Actual Result

Hotkeys Z, PageUp, PageDown are not working, nothing happens at the viewer page.

Expected Result

1. When user hits hotkey Z, the Zoom tool should be activated at the toolbar
2. User should be able to navigate on the thumbnails using hotkeys: PageUp and PageDown 2.1. When user hits hotkey PageDown, the next thumbnail series should be loaded in the viewport 2.2. When user hits hotkey PageUp, the previous thumbnail series should be loaded in the viewport

Issue 2 - Description:

When there are multiple viewports, hotkey = does not reset the inverted status of the image in the active viewport.

In the gif below, user presses the = hotkey to reset all viewports before clicking on the Reset button

Steps to Reproduce

1. Open a study
2. Change layout to 3 viewports
3. Invert the image by pressing hotkey I
4. Press hotkey =

Actual Result

Hotkey = does not reset the inverted status of the image. If user clicks on the Reset button, the inverted image will be reset with success.

Expected Result

User should be able to reset the inverted image (in the active viewport) by pressing hotkey = or clicking on Reset button on toolbar.

[galelis]

@mirnasilva We actually have 3 issues here.

• Z hotkey not working Fixed, we were missing the command setZoomTool.

• PageUp, PageDown hotkey not working Fixed, we have never implemented the method to navigate through current displaySets

• = hotkey not resetting inverted configurations. Wht happens is that = is current attached to zoom to fit so it’s not suposed to reset invert. We have space currently set for Reset and it was properly resetting the invert configurations. If you can take a look again and let me know if I tested it wrongly.

Fixable issues should be fixed by the PR: #1446

[dannyrb]

👋 @cmcgee-mac

We’re trying to document the available commands better. The end-goal is to have them listed in the readme for our maintained extensions. You can see a few listed here:

https://github.com/OHIF/Viewers/tree/master/extensions/cornerstone#commands-module

Those might be out of date. You can always dig in to the source and see the list of named commands in the extension’s command definitions:

https://github.com/OHIF/Viewers/blob/master/extensions/cornerstone/src/commandsModule.js#L167-L246

---

It looks like, at some point, we switched from setZoomTool to a more generic setToolActive that takes a tool’s name as a parameter:

https://github.com/OHIF/Viewers/blob/master/extensions/cornerstone/src/commandsModule.js#L84-L89

We are more than happy to hear any thoughts you have around improving discovery of commands, or PRs to improve our documentation ^_^