sul-dlss-deprecated / iiifManifestLayouts

Other
10 stars 5 forks source link

Remove thumbnail when any image is selected #119

Closed nein09 closed 8 years ago

nein09 commented 8 years ago

This is a way of addressing https://github.com/sul-dlss/iiifManifestLayouts/issues/118

The behavior there is a result of the fact that the thumbnail was only removed when canvas.openMainTileSource() was called. That, in turn, was only ever called when a canvas was clicked (because semantic zoom isn't in yet).

That means that there was a set of checkboxes next to the image, which users could click on in order to load full-fidelity images, but the thumbnail wouldn't go away, because the checkboxes themselves were wired up directly to the image objects. In this way, a user could cause the main tile source to get opened, without canvas.openMainTileSource() ever having been called.

The change is to remove the thumbnail when one of those boxes is checked and an image is opened.

nein09 commented 8 years ago

I should note that if we want the main tile source to always be displayed with no option of turning it off, I might do this differently.