ReferenceStrip: Fixed destroy() exception and made element focusable

This commit is contained in:
Mark Salsbery 2021-03-17 13:04:13 -07:00
parent 16ab337146
commit e8accfdfaf
2 changed files with 7 additions and 1 deletions

View File

@ -41,6 +41,8 @@ OPENSEADRAGON CHANGELOG
* Added setImageFormatsSupported function (#1954 @pandaxtc) * Added setImageFormatsSupported function (#1954 @pandaxtc)
* Added dragToPan to the GestureSettings class, implemented in Viewer (#1956 @msalsbery) * Added dragToPan to the GestureSettings class, implemented in Viewer (#1956 @msalsbery)
* Added preventDefault option to MouseTracker handlers: scrollHandler, keyDownHandler, keyUpHandler, keyHandler (#1957 @msalsbery) * Added preventDefault option to MouseTracker handlers: scrollHandler, keyDownHandler, keyUpHandler, keyHandler (#1957 @msalsbery)
* ReferenceStrip: Fixed issue where its element was being removed from its parent element twice on destroy, causing an exception (#1957 @msalsbery)
* ReferenceStrip: Made its element focusable for keyboard navigation (#1957 @msalsbery)
2.4.2: 2.4.2:

View File

@ -96,6 +96,8 @@ $.ReferenceStrip = function ( options ) {
this.minPixelRatio = this.viewer.minPixelRatio; this.minPixelRatio = this.viewer.minPixelRatio;
this.element.tabIndex = 0;
style = this.element.style; style = this.element.style;
style.marginTop = '0px'; style.marginTop = '0px';
style.marginRight = '0px'; style.marginRight = '0px';
@ -281,7 +283,7 @@ $.ReferenceStrip.prototype = {
this.tracker.destroy(); this.tracker.destroy();
if (this.element) { if (this.element) {
this.element.parentNode.removeChild(this.element); this.viewer.removeControl( this.element );
} }
} }
@ -305,6 +307,8 @@ function onStripClick( event ) {
this.viewer.goToPage( page ); this.viewer.goToPage( page );
} }
this.element.focus();
} }