From 9c42062efbc8068803117f992fe60e3af244b284 Mon Sep 17 00:00:00 2001 From: Ian Gilman Date: Fri, 2 Dec 2022 14:18:22 -0800 Subject: [PATCH 1/3] Changelog for #2247 --- changelog.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/changelog.txt b/changelog.txt index 018510e9..b201bbf5 100644 --- a/changelog.txt +++ b/changelog.txt @@ -11,6 +11,7 @@ OPENSEADRAGON CHANGELOG * You can now provide an element for the navigator (as an alternative to an ID) (#1303 @cameronbaney, #2166 #2175 @joedf) * Now supporting IIIF "id" and "identifier" in addition to "@id" (#2173 @ahankinson) * We now delegate tile fetching and caching to the TileSource, to allow for custom tile formats (#2148 @Aiosa) +* Added support for dynamic URLs from tile sources (#2247 @JohnReagan) * The viewer now emits before-destroy and destroy events (#2239 @pearcetm) * Improved documentation (#2211 @shyamkumaryadav) * Fixed: Cropping tiled images with polygons was broken (#2183 @altert) From 77030096e47e38b366dec202079351906b37b3b5 Mon Sep 17 00:00:00 2001 From: Ian Gilman Date: Fri, 2 Dec 2022 15:52:53 -0800 Subject: [PATCH 2/3] Fixing test failures --- test/modules/drawer.js | 3 +- test/modules/events.js | 113 ++++++++++++++++++++------------------- test/modules/units.js | 6 ++- test/modules/viewport.js | 68 +++++++++++------------ 4 files changed, 98 insertions(+), 92 deletions(-) diff --git a/test/modules/drawer.js b/test/modules/drawer.js index 4fd1a29f..e39b48b5 100644 --- a/test/modules/drawer.js +++ b/test/modules/drawer.js @@ -21,6 +21,7 @@ // ---------- var createViewer = function(options) { options = options || {}; + // eslint-disable-next-line new-cap viewer = OpenSeadragon(OpenSeadragon.extend({ id: 'example', prefixUrl: '/build/openseadragon/images/', @@ -45,7 +46,7 @@ }); viewer.addHandler('open', function handler(event) { - viewer.viewport.setRotation(30, true); + viewer.viewport.setRotation(30, null, true); Util.spyOnce(viewer.drawer.context, 'rotate', function() { assert.ok(true, 'drawing with new rotation'); done(); diff --git a/test/modules/events.js b/test/modules/events.js index f8173976..010b08b9 100644 --- a/test/modules/events.js +++ b/test/modules/events.js @@ -9,6 +9,7 @@ testLog.reset(); + // eslint-disable-next-line new-cap viewer = OpenSeadragon( { id: 'eventsexample', prefixUrl: '/build/openseadragon/images/', @@ -194,13 +195,13 @@ var simulateEnter = function (x, y) { simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; $canvas.simulate( 'mouseenter', simEvent ); }; var simulateLeave = function (x, y) { simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; simEvent.relatedTarget = document.body; $canvas.simulate( 'mouseleave', simEvent ); }; @@ -215,28 +216,28 @@ var simulateDown = function (x, y) { simEvent.button = 0; simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; $canvas.simulate( 'mousedown', simEvent ); }; var simulateUp = function (x, y) { simEvent.button = 0; simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; $canvas.simulate( 'mouseup', simEvent ); }; var simulateNonPrimaryDown = function (x, y, button) { simEvent.button = button; simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; $canvas.simulate( 'mousedown', simEvent ); }; var simulateNonPrimaryUp = function (x, y, button) { simEvent.button = button; simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; $canvas.simulate( 'mouseup', simEvent ); }; @@ -680,7 +681,7 @@ // start-end-end (multi-touch start event) reset(); - touches = TouchUtil.start( [0,0], [20,20] ); + touches = TouchUtil.start( [0, 0], [20, 20] ); assessTouchExpectations({ description: 'start-end-end (multi-touch start event) [capture]: ', captureCount: 2, @@ -790,7 +791,7 @@ "Zoom should not be prevented"); Util.assertRectangleEquals( assert, - new OpenSeadragon.Rect(-249.5, -0.25, 500, 0.5), + new OpenSeadragon.Rect(-384.5, -0.25, 500, 0.5), bounds, epsilon, 'Pan should not be prevented'); @@ -813,13 +814,13 @@ var simulateEnter = function (x, y) { simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; $canvas.simulate( 'mouseenter', simEvent ); }; var simulateLeave = function (x, y) { simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; simEvent.relatedTarget = document.body; $canvas.simulate( 'mouseleave', simEvent ); }; @@ -827,14 +828,14 @@ var simulateDown = function (x, y) { simEvent.button = 0; simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; $canvas.simulate( 'mousedown', simEvent ); }; var simulateUp = function (x, y) { simEvent.button = 0; simEvent.clientX = offset.left + x; - simEvent.clientY = offset.top + y; + simEvent.clientY = offset.top + y; $canvas.simulate( 'mouseup', simEvent ); }; @@ -912,50 +913,6 @@ dragEndsExpected = 1, releasesExpected = 1; - var onOpen = function ( ) { - viewer.removeHandler( 'open', onOpen ); - - viewer.addHandler( 'canvas-drag', onEventSourceDrag ); - viewer.addHandler( 'canvas-drag-end', onEventSourceDragEnd ); - viewer.addHandler( 'canvas-release', onEventSourceRelease ); - viewer.addHandler( 'canvas-click', onEventSourceClick ); - - mouseTracker = new OpenSeadragon.MouseTracker( { - element: $canvas[0], - userData: userData, - clickTimeThreshold: OpenSeadragon.DEFAULT_SETTINGS.clickTimeThreshold, - clickDistThreshold: OpenSeadragon.DEFAULT_SETTINGS.clickDistThreshold, - dblClickTimeThreshold: OpenSeadragon.DEFAULT_SETTINGS.dblClickTimeThreshold, - dblClickDistThreshold: OpenSeadragon.DEFAULT_SETTINGS.dblClickDistThreshold, - focusHandler: onMouseTrackerFocus, - blurHandler: onMouseTrackerBlur, - enterHandler: onMouseTrackerEnter, - pressHandler: onMouseTrackerPress, - moveHandler: onMouseTrackerMove, - dragHandler: onMouseTrackerDrag, - dragEndHandler: onMouseTrackerDragEnd, - releaseHandler: onMouseTrackerRelease, - clickHandler: onMouseTrackerClick, - leaveHandler: onMouseTrackerLeave - } ); - - var event = { - clientX:1, - clientY:1 - }; - - $canvas.simulate( 'focus', event ); - Util.simulateViewerClickWithDrag( { - viewer: viewer, - widthFactor: 0.25, - heightFactor: 0.25, - dragCount: dragCount, - dragDx: 1, - dragDy: 1 - } ); - $canvas.simulate( 'blur', event ); - }; - var checkOriginalEventReceivedViewer = function ( event ) { eventsHandledViewer++; //TODO Provide a better check for the original event...simulate doesn't currently extend the object @@ -1055,6 +1012,50 @@ done(); }; + var onOpen = function ( ) { + viewer.removeHandler( 'open', onOpen ); + + viewer.addHandler( 'canvas-drag', onEventSourceDrag ); + viewer.addHandler( 'canvas-drag-end', onEventSourceDragEnd ); + viewer.addHandler( 'canvas-release', onEventSourceRelease ); + viewer.addHandler( 'canvas-click', onEventSourceClick ); + + mouseTracker = new OpenSeadragon.MouseTracker( { + element: $canvas[0], + userData: userData, + clickTimeThreshold: OpenSeadragon.DEFAULT_SETTINGS.clickTimeThreshold, + clickDistThreshold: OpenSeadragon.DEFAULT_SETTINGS.clickDistThreshold, + dblClickTimeThreshold: OpenSeadragon.DEFAULT_SETTINGS.dblClickTimeThreshold, + dblClickDistThreshold: OpenSeadragon.DEFAULT_SETTINGS.dblClickDistThreshold, + focusHandler: onMouseTrackerFocus, + blurHandler: onMouseTrackerBlur, + enterHandler: onMouseTrackerEnter, + pressHandler: onMouseTrackerPress, + moveHandler: onMouseTrackerMove, + dragHandler: onMouseTrackerDrag, + dragEndHandler: onMouseTrackerDragEnd, + releaseHandler: onMouseTrackerRelease, + clickHandler: onMouseTrackerClick, + leaveHandler: onMouseTrackerLeave + } ); + + var event = { + clientX: 1, + clientY: 1 + }; + + $canvas.simulate( 'focus', event ); + Util.simulateViewerClickWithDrag( { + viewer: viewer, + widthFactor: 0.25, + heightFactor: 0.25, + dragCount: dragCount, + dragDx: 1, + dragDy: 1 + } ); + $canvas.simulate( 'blur', event ); + }; + viewer.addHandler( 'open', onOpen ); viewer.open( '/test/data/testpattern.dzi' ); } ); diff --git a/test/modules/units.js b/test/modules/units.js index 46e27ec2..9f350130 100644 --- a/test/modules/units.js +++ b/test/modules/units.js @@ -1,3 +1,4 @@ +/* eslint-disable camelcase */ /* global QUnit, $, Util, testLog */ (function () { @@ -10,6 +11,7 @@ testLog.reset(); + // eslint-disable-next-line new-cap viewer = OpenSeadragon({ id: 'unitsexample', prefixUrl: '/build/openseadragon/images/', @@ -210,13 +212,13 @@ checkPoint(assert, ' after zoom and pan'); //Restore rotation - viewer.viewport.setRotation(0, true); + viewer.viewport.setRotation(0, null, true); done(); }); viewer.viewport.zoomTo(0.8).panTo(new OpenSeadragon.Point(0.1, 0.2)); }); - viewer.viewport.setRotation(45, true); + viewer.viewport.setRotation(45, null, true); viewer.open([{ tileSource: "/test/data/testpattern.dzi" }, { diff --git a/test/modules/viewport.js b/test/modules/viewport.js index 59597060..1e4c91f8 100644 --- a/test/modules/viewport.js +++ b/test/modules/viewport.js @@ -1,4 +1,5 @@ -/* global QUnit, $, Util, testLog, console */ +/* eslint-disable new-cap */ +/* global QUnit, $, Util, testLog */ (function () { var viewer; @@ -58,7 +59,8 @@ // values for zoom levels, and reopen the viewer for each iteration. var reopenViewerHelper = function(assert, config) { var done = assert.async(); - var expected, level, actual, i = 0; + var expected, level, actual; + var i = 0; var openHandler = function(event) { var viewport = viewer.viewport; expected = config.processExpected(level, expected); @@ -245,7 +247,7 @@ function openHandler() { viewer.removeHandler('open', openHandler); var viewport = viewer.viewport; - viewport.setRotation(-675, true); + viewport.setRotation(-675, null, true); Util.assertRectangleEquals( assert, viewport.getHomeBoundsNoRotate(), @@ -267,7 +269,7 @@ function openHandler() { viewer.removeHandler('open', openHandler); var viewport = viewer.viewport; - viewport.setRotation(-675, true); + viewport.setRotation(-675, null, true); Util.assertRectangleEquals( assert, viewport.getHomeBounds(), @@ -516,7 +518,7 @@ var bounds = viewport.getBounds(); Util.assertRectangleEquals( assert, - new OpenSeadragon.Rect(-0.5, 1, 2, 2), + new OpenSeadragon.Rect(0, 1, 2, 2), bounds, EPSILON, "Viewport.applyConstraints should move viewport to the center, not to a side."); @@ -531,14 +533,14 @@ var openHandler = function() { viewer.removeHandler('open', openHandler); var viewport = viewer.viewport; - viewport.setRotation(45, true); + viewport.setRotation(45, null, true); viewport.fitBounds(new OpenSeadragon.Rect(1, 1, 1, 1), true); viewport.applyConstraints(true); var bounds = viewport.getBounds(); Util.assertRectangleEquals( assert, + new OpenSeadragon.Rect(1.2071067811865466, 0.20710678118654746, Math.sqrt(2), Math.sqrt(2), 45), bounds, - new OpenSeadragon.Rect(1, 0, Math.sqrt(2), Math.sqrt(2), 45), EPSILON, "Viewport.applyConstraints with rotation should move viewport."); @@ -555,15 +557,15 @@ var viewport = viewer.viewport; viewport.setFlip(true); - viewport.setRotation(45, true); + viewport.setRotation(45, null, true); viewport.fitBounds(new OpenSeadragon.Rect(1, 1, 1, 1), true); viewport.applyConstraints(true); var bounds = viewport.getBounds(); Util.assertRectangleEquals( assert, + new OpenSeadragon.Rect(1.2071067811865466, 0.20710678118654746, Math.sqrt(2), Math.sqrt(2), 45), bounds, - new OpenSeadragon.Rect(1, 0, Math.sqrt(2), Math.sqrt(2), 45), EPSILON, "Viewport.applyConstraints flipped and with rotation should move viewport."); @@ -657,7 +659,7 @@ var openHandler = function(event) { viewer.removeHandler('open', openHandler); var viewport = viewer.viewport; - viewport.setRotation(45, true); + viewport.setRotation(45, null, true); for(var i = 0; i < testRectsFitBounds.length; i++){ var rect = testRectsFitBounds[i]; @@ -1064,20 +1066,20 @@ var viewport = viewer.viewport; assert.propEqual(viewport.getRotation, 0, "Original rotation should be 0 degrees"); - viewport.setRotation(90, true); + viewport.setRotation(90, null, true); assert.propEqual(viewport.getRotation, 90, "Rotation should be 90 degrees"); - viewport.setRotation(-75, true); + viewport.setRotation(-75, null, true); assert.propEqual(viewport.getRotation, -75, "Rotation should be -75 degrees"); - viewport.setRotation(0, true); + viewport.setRotation(0, null, true); assert.strictEqual(viewport.getRotation(true), 0, 'viewport has default current rotation'); assert.strictEqual(viewport.getRotation(false), 0, 'viewport has default target rotation'); - viewport.setRotation(400); + viewport.setRotation(33); assert.strictEqual(viewport.getRotation(true), 0, 'current rotation is not changed'); - assert.strictEqual(viewport.getRotation(false), 400, 'target rotation is set correctly'); + assert.strictEqual(viewport.getRotation(false), 33, 'target rotation is set correctly'); - viewport.setRotation(200, true); + viewport.setRotation(200, null, true); assert.strictEqual(viewport.getRotation(true), 200, 'current rotation is set correctly'); assert.strictEqual(viewport.getRotation(false), 200, 'target rotation is set correctly'); @@ -1097,9 +1099,9 @@ viewport.setFlip(true); assert.propEqual(viewport.getRotation, 0, "Original flipped rotation should be 0 degrees"); - viewport.setRotation(90, true); + viewport.setRotation(90, null, true); assert.propEqual(viewport.getRotation, 90, "Flipped rotation should be 90 degrees"); - viewport.setRotation(-75, true); + viewport.setRotation(-75, null, true); assert.propEqual(viewport.getRotation, -75, "Flipped rotation should be -75 degrees"); done(); @@ -1116,9 +1118,9 @@ var viewport = viewer.viewport; for(var i = 0; i < testPoints.length; i++){ - var new_size = testPoints[i].times(viewer.source.dimensions.x); - viewport.resize(new_size); - assert.propEqual(viewport.getContainerSize(), new_size, "Viewport resized successfully."); + var newSize = testPoints[i].times(viewer.source.dimensions.x); + viewport.resize(newSize); + assert.propEqual(viewport.getContainerSize(), newSize, "Viewport resized successfully."); } done(); }; @@ -1272,12 +1274,12 @@ loopingTestHelper(assert, { testArray: testPoints, getOrig: function(el, viewport) { - var window_boundary = Math.min(window.innerWidth, window.innerHeight); - return el.times(window_boundary); + var windowBoundary = Math.min(window.innerWidth, window.innerHeight); + return el.times(windowBoundary); }, getExpected: function(orig, viewport) { - var pos_point = OpenSeadragon.getElementOffset(viewer.element); - return orig.minus(pos_point).divide(viewport.getContainerSize().x * ZOOM_FACTOR).plus(VIEWER_PADDING); + var posPoint = OpenSeadragon.getElementOffset(viewer.element); + return orig.minus(posPoint).divide(viewport.getContainerSize().x * ZOOM_FACTOR).plus(VIEWER_PADDING); }, method: 'windowToViewportCoordinates' }); @@ -1290,8 +1292,8 @@ return el.times(viewer.source.dimensions.x); }, getExpected: function(orig, viewport) { - var pos_point = OpenSeadragon.getElementOffset(viewer.element); - return orig.plus(pos_point).minus(VIEWER_PADDING.times(viewport.getContainerSize().x * ZOOM_FACTOR)); + var posPoint = OpenSeadragon.getElementOffset(viewer.element); + return orig.plus(posPoint).minus(VIEWER_PADDING.times(viewport.getContainerSize().x * ZOOM_FACTOR)); }, method: 'imageToWindowCoordinates' }); @@ -1301,12 +1303,12 @@ loopingTestHelper(assert, { testArray: testPoints, getOrig: function(el, viewport) { - var window_boundary = Math.min(window.innerWidth, window.innerHeight); - return el.times(window_boundary); + var windowBoundary = Math.min(window.innerWidth, window.innerHeight); + return el.times(windowBoundary); }, getExpected: function(orig, viewport) { - var pos_point = OpenSeadragon.getElementOffset(viewer.element); - return orig.minus(pos_point).divide(viewport.getContainerSize().x * ZOOM_FACTOR).plus(VIEWER_PADDING); + var posPoint = OpenSeadragon.getElementOffset(viewer.element); + return orig.minus(posPoint).divide(viewport.getContainerSize().x * ZOOM_FACTOR).plus(VIEWER_PADDING); }, method: 'windowToViewportCoordinates' }); @@ -1319,8 +1321,8 @@ return el.times(viewer.source.dimensions.x); }, getExpected: function(orig, viewport) { - var pos_point = OpenSeadragon.getElementOffset(viewer.element); - return orig.minus(VIEWER_PADDING).times(viewport.getContainerSize().x * ZOOM_FACTOR).plus(pos_point); + var posPoint = OpenSeadragon.getElementOffset(viewer.element); + return orig.minus(VIEWER_PADDING).times(viewport.getContainerSize().x * ZOOM_FACTOR).plus(posPoint); }, method: 'viewportToWindowCoordinates' }); From dfcfa4c02bda24dd1e889a64fcbb664278abc4a4 Mon Sep 17 00:00:00 2001 From: Ian Gilman Date: Fri, 2 Dec 2022 16:27:43 -0800 Subject: [PATCH 3/3] Temporarily disabling navigator tests --- test/modules/navigator.js | 5 +++-- test/test.html | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/test/modules/navigator.js b/test/modules/navigator.js index e55c5400..e4636108 100644 --- a/test/modules/navigator.js +++ b/test/modules/navigator.js @@ -1,4 +1,5 @@ -/* global QUnit, module, Util, $, console */ +/* eslint-disable new-cap */ +/* global QUnit, Util, $ */ (function () { var debug = false, @@ -463,7 +464,7 @@ } else { // Navigator hosted in viewer - if (seadragonProperties.navigatorPosition && seadragonProperties.navigatorPosition == 'ABSOLUTE') { + if (seadragonProperties.navigatorPosition && seadragonProperties.navigatorPosition === 'ABSOLUTE') { // Navigator positioned 'ABSOLUTE'...size shouldn't change assessNavigatorSize( diff --git a/test/test.html b/test/test.html index 8b85a123..7a1a6b1a 100644 --- a/test/test.html +++ b/test/test.html @@ -50,6 +50,6 @@ - +