feat(KeyboardShortcut): Add left and right arrow to cycle through images

[lchauvin]

Related to #610

[netlify]

✅ Deploy Preview for volview-dev ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	e92dd52
🔍 Latest deploy log	https://app.netlify.com/sites/volview-dev/deploys/6707ff3f45f618000803180a
😎 Deploy Preview	https://deploy-preview-611--volview-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[PaulHax]

Works good. Thanks!

[floryst]

This does have an issue where not-yet reconstructed DICOM images aren't included in the cycling. I'm unifying the image list in #606, which should make this easier.

[lchauvin]

Has the list of images been unified ?

[PaulHax]

@lchauvin #606 added an imageCacheStore which holds image IDs for DICOM and other image types, even as they are loaded. So the arrow keys probably want to loop through the imageCacheStore.imageIds array.

[floryst]

I've given this some thought, so here are some ideas I had for this feature:

• the patient studies + images sidebar should have a backing store for it, which defines the order of the seen images. The order of the images in the image cache store might not be particularly useful, since the order still might not be what is visually seen in the sidebar
• The left/right arrow cycles will use this backing store to select the next image

[lchauvin]

Should I wait for the backing store before making any modification ?

[floryst]

Revisiting this PR: we haven't had time to implement this. The idea (if you want to take it on) is to take the logic in ImageDataBrowser + PatientBrowser and move that into a store.