From f567fbfeaf3e5625e79cc87bb0b1c514db182090 Mon Sep 17 00:00:00 2001 From: Imran Asghar Date: Thu, 13 Aug 2020 11:50:49 -0400 Subject: [PATCH 1/2] Replace v2 iiifSize max with full Size parameter is set to max when the tile width and image width are same. But, in version 2, max is not required to be supported by IIIF v2 services on any compliance level. --- src/iiiftilesource.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/iiiftilesource.js b/src/iiiftilesource.js index 1c0adc13..76479e76 100644 --- a/src/iiiftilesource.js +++ b/src/iiiftilesource.js @@ -398,7 +398,7 @@ $.extend( $.IIIFTileSource.prototype, $.TileSource.prototype, /** @lends OpenSea } if ( levelWidth < tileWidth && levelHeight < tileHeight ){ if ( this.version === 2 && levelWidth === this.width ) { - iiifSize = "max"; + iiifSize = "full"; } else if ( this.version === 3 && levelWidth === this.width && levelHeight === this.height ) { iiifSize = "max"; } else if ( this.version === 3 ) { @@ -420,7 +420,7 @@ $.extend( $.IIIFTileSource.prototype, $.TileSource.prototype, /** @lends OpenSea iiifSizeW = Math.ceil( iiifTileW * scale ); iiifSizeH = Math.ceil( iiifTileH * scale ); if ( this.version === 2 && iiifSizeW === this.width ) { - iiifSize = "max"; + iiifSize = "full"; } else if ( this.version === 3 && iiifSizeW === this.width && iiifSizeH === this.height ) { iiifSize = "max"; } else if (this.version === 3) { From 4ab8cc748188f6c4dc5ad7d23e91945dd0194173 Mon Sep 17 00:00:00 2001 From: Ian Gilman Date: Mon, 24 Aug 2020 17:39:45 -0700 Subject: [PATCH 2/2] Change log for #1871 --- changelog.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/changelog.txt b/changelog.txt index 017d4e55..d2e5f249 100644 --- a/changelog.txt +++ b/changelog.txt @@ -8,6 +8,7 @@ OPENSEADRAGON CHANGELOG * Better cleanup on destruction, to avoid memory leaks (#1832 @JoFrMueller) * Miscellaneous code cleanup (#1840 @msalsbery) * You can now specify tileSize for the Zoomify Tile Source (#1868 @abrlam) +* Better use of IIIF "max" and "full" URL parameters (#1871 @MImranAsghar) 2.4.2: