Commit Graph

277 Commits

Author SHA1 Message Date
Ian Gilman
1c6deafe8b Added viewport property to Viewer documentation 2013-05-02 09:57:03 -07:00
Benjamin Gilbert
3652c7066c Re-add viewer.source check in updateMulti()
It should be redundant in normal operation, but may prevent a timer leak
in case of a bug in the open/close path.
2013-04-29 13:17:37 -04:00
Benjamin Gilbert
8cb2714dae Fix timer leak after multiple Viewer.open() calls, take 2
Track the request ID of the outstanding animation frame and cancel it
on Viewer.close().

Fixes #76.
2013-04-26 22:32:51 -04:00
Benjamin Gilbert
23c20e3d5a Revert "Fix timer leak after multiple Viewer.open() calls"
This should be done with cancelAnimationFrame() instead.

This reverts commit 47aba60940.
2013-04-26 21:48:48 -04:00
Heath Nielson
d16eb04a8a Fix setStrings() with the same fix as getStrings(). Initialize container to
I18N.
2013-04-26 15:18:27 -06:00
Benjamin Gilbert
47aba60940 Fix timer leak after multiple Viewer.open() calls
Keep a counter of Viewer.close() calls in private state and a copy in
the timer's closure, and stop the timer when they no longer match.

Fixes #76.
2013-04-25 00:37:36 -04:00
Benjamin Gilbert
9eb19d6c9c Garbage-collect strings 2013-04-25 00:23:02 -04:00
Benjamin Gilbert
b315b148c2 Drop remains of code to show "Loading..." message
It calls a method that no longer exists, but only when an impossible
condition is true.
2013-04-24 23:57:35 -04:00
Benjamin Gilbert
f958e66f05 Fix typo in preserveViewport handling 2013-04-23 23:58:58 -04:00
houseofyin
bbf0fc4302 Updated to address feedbac from @iangilman 2013-04-04 22:30:59 -04:00
Heath Nielson
530e445756 Fix a problem with getString when the string property is a sub-property. 2013-04-03 12:33:47 -06:00
houseofyin
ccf8934d0d Backout unneeded change in mousetracker.js 2013-04-03 13:42:16 -04:00
houseofyin
4c126dd39a Trying again to backout button.js changes 2013-04-03 13:36:55 -04:00
houseofyin
b352fe711b Trying again to backout button.js changes 2013-04-03 13:33:39 -04:00
houseofyin
df3344f67c Getting into alignment with code style guidelines 2013-04-03 11:51:24 -04:00
houseofyin
b3d358fa35 First pass at bringing the code in compliance with the project code standards 2013-04-02 21:07:10 -04:00
houseofyin
569a816df4 Backing out an event handler change 2013-04-02 21:06:07 -04:00
houseofyin
cfd5e67462 Backing out an event handler change 2013-04-02 18:40:28 -04:00
houseofyin
5bf4b51925 Checkpoint. Anchoring click to middle of view. Adding call to apply to constraints on viewer after the click, and additional testing 2013-04-01 22:33:58 -04:00
Chris Adams
2345f9b217 DziTileSource: avoid changing relative paths
For cases where DZI files aren't explicitly configured with a tilesUrl,
the logic for generating a URL from the DZI source URL would inject an
extra leading `/` which would either cause an unnecessary redirect or
break depending on whether the webserver in use attempts to normalize
`//` to `/`.

This change also removes some URL processing logic which duplicates the
browser's default behaviour
2013-03-26 16:11:27 -04:00
thatcher
9aecfddcbf refs #54 - added missing support for tilesource configuration as xml string or json string via Viewer.open 2013-03-26 14:35:43 -04:00
thatcher
86f101f02c refs #54 - pointing deprecated method at correct current method. 2013-03-26 09:25:56 -04:00
thatcher
266f33094c addresses #54, dont use openDZI, use open. openDZI now calls open. all the details of tileHost were already handled in DZITileSource.configure. $.createFromDZI is fully deprecated and will throw an error. 2013-03-26 09:02:34 -04:00
houseofyin
8db1a15282 A little cleanup of the border issue 2013-03-23 23:06:34 -04:00
houseofyin
450760d3b7 Merge branch 'master' of github.com:openseadragon/openseadragon into issue37-navigator
Conflicts:
	test/test.html
2013-03-23 21:44:22 -04:00
houseofyin
a7cec524eb Tweaked the border handling a bit 2013-03-23 21:11:30 -04:00
houseofyin
325bae2b6f Checking in a fix for issue 48 2013-03-21 13:54:39 -04:00
houseofyin
7187bca4e7 Merge branch 'master' of github.com:openseadragon/openseadragon into issue37-navigator 2013-03-18 14:38:10 -04:00
houseofyin
80ff8f908a Another checkpoint, added keyboard navigation to the main viewer 2013-03-16 16:35:33 -04:00
houseofyin
3e2b337a52 Another checkpoint 2013-03-15 22:29:27 -04:00
houseofyin
3b2bde2940 Initial pass at making the navigator float 2013-03-15 10:59:47 -04:00
thatcher
0297a20aee merged conflict 2013-03-14 09:08:27 -04:00
thatcher
10f60c88cb minImageRatio should never be set below 1 unless you want to load the Nth level tiles to level N-M, which effectively throttles network requests and draw animations but does not produce a better picture 2013-03-14 08:12:01 -04:00
thatcher
0b75a2f259 tweaking default setting to find optimal performance point while preserving responsivness 2013-03-14 07:22:27 -04:00
thatcher
f0c7870f4e cleaning up new setting as applied to collections 2013-03-14 07:22:26 -04:00
thatcher
b9741daaaf removing debug console log statement per ventero's review notes 2013-03-14 07:22:26 -04:00
thatcher
537ea0b844 simplifying zoom settings so scroll and click arent significantly different 2013-03-14 07:22:26 -04:00
thatcher
464cf076a6 simplifying zoom settings so scroll and click arent significantly different 2013-03-14 07:22:26 -04:00
thatcher
a5421917a4 minZoomImageRatio should not be less than minPixelRatio to avoid blurry full zoom out. Its worth noting now that several options are 'coupled' but dont have logic which addresses the coupling so as to avoid ill affects from settings that dont make sense for how they are coupled. 2013-03-14 07:22:26 -04:00
thatcher
86e27c18f7 what was initally a feature branch to work on just canvas prerender, evolved into a feature branch focused on broader ideas discussed in issue #4 - I've basically been able to reduce time spent in drawTile by half. good stuff, thanks to dustmoo for getting us looking into this 2013-03-14 07:22:26 -04:00
thatcher
fe0cafea21 improved caching follows existing use of tile.unload to delete prerendered canvas 2013-03-14 07:22:26 -04:00
thatcher
2be165fa70 niave implementation of prerender for canvas, the first optimization discussed here: http://www.html5rocks.com/en/tutorials/canvas/performance/ 2013-03-14 07:22:25 -04:00
thatcher
417a93d5ae Merge branch 'master' of https://github.com/openseadragon/openseadragon 2013-03-12 20:15:36 -04:00
thatcher
b3146683f2 applying patch provided by eikeon for position parameter constructor of TileSource. At some point I hope to deprecate most of these constructors that have more than two positional parameters. 2013-03-12 20:12:08 -04:00
Ian Gilman
b4ddb5a953 Improved build banners, including making them visible in minified version 2013-03-11 11:52:30 -07:00
Chris Thatcher
c164e8fac5 Merge pull request #24 from openseadragon/fullscreen
Fullscreen native javascript support now available. will increment build id and publish.  Thanks ventero and iangilman for your reviews.
2013-03-07 20:57:11 -08:00
thatcher
5203cc14b0 removing extra whitespace per pull request review 2013-03-07 23:50:27 -05:00
thatcher
f107aaf0e8 removing debug console log statement per ventero's review notes 2013-03-06 21:53:20 -05:00
thatcher
6285a779f3 cleaning up event names, reducing demo cases to just addHandler interface, normalizing event data to express more idiomatic javascript pattern of passing object instead of invoking callback with positional parameters 2013-03-06 17:34:12 -05:00
thatcher
3801326e40 pulling full screen api detection out into is own file per ians suggestion 2013-03-06 15:36:52 -05:00
thatcher
75f7589312 simplifying zoom settings so scroll and click arent significantly different 2013-03-06 07:43:12 -05:00
thatcher
0bdc4383bd simplifying zoom settings so scroll and click arent significantly different 2013-03-06 07:35:47 -05:00
thatcher
e04813e477 minZoomImageRatio should not be less than minPixelRatio to avoid blurry full zoom out. Its worth noting now that several options are 'coupled' but dont have logic which addresses the coupling so as to avoid ill affects from settings that dont make sense for how they are coupled. 2013-03-06 06:44:55 -05:00
thatcher
096b02d1c8 addressing typo in comment pointed out by @iangilian 2013-03-06 06:03:05 -05:00
thatcher
9503fac39d merging, not sure why I get a conflict on this, must be doing something wrong in my work flow 2013-03-06 05:55:53 -05:00
thatcher
61a844bdc0 what was initally a feature branch to work on just canvas prerender, evolved into a feature branch focused on broader ideas discussed in issue #4 - I've basically been able to reduce time spent in drawTile by half. good stuff, thanks to dustmoo for getting us looking into this 2013-03-06 05:51:31 -05:00
thatcher
11a0f3b315 improved caching follows existing use of tile.unload to delete prerendered canvas 2013-03-05 07:30:37 -05:00
houseofyin
a432a38d3d 23 Initial attempt a this feature along with a demo page 2013-03-04 15:19:35 -05:00
thatcher
2fc6cc3876 niave implementation of prerender for canvas, the first optimization discussed here: http://www.html5rocks.com/en/tutorials/canvas/performance/ 2013-03-01 09:23:47 -05:00
Chris Thatcher
fc479b5ecb Merge pull request #26 from openseadragon/memoizelevelscale
memoize default implementation of getLevelScale
2013-03-01 06:18:08 -08:00
thatcher
274df71c0a refs #22 use Math.pow per houseofyin and memoize implementation of getLevelScale per acdha 2013-03-01 08:42:06 -05:00
thatcher
6d4d7cc8c6 niave implementation of prerender for canvas, the first optimization discussed here: http://www.html5rocks.com/en/tutorials/canvas/performance/ 2013-03-01 08:14:35 -05:00
thatcher
b560a06c30 correcting comment per ventero's review 2013-02-28 16:19:04 -05:00
thatcher
e56632c96f blendTime needs to be equal to animationTime to avoid blurred images 2013-02-28 15:56:50 -05:00
thatcher
2819ff14e5 reverting a couple changes to core openseadragon options unrelated to issue #2 . will explore these in #4 were they have already been referenced. 2013-02-28 15:55:24 -05:00
thatcher
807cdd337c cant set blendTime to 0, causes strange behavior in navigator and lip tile sources. should have made the basic config changes in #2 and #3 in a diferent branch related to #4. will do before pulling 2013-02-28 15:55:24 -05:00
thatcher
6d3ddd8095 by feel, best performance, changed immediateRender to true, springStiffness to 7 - issue #2 2013-02-28 15:55:24 -05:00
thatcher
5b6a2f5873 basic implementation of issue #2 2013-02-28 15:55:24 -05:00
thatcher
5dd8798f7f addressing venturo's review notes 2013-02-28 15:34:44 -05:00
thatcher
a4477cd765 better encapsulation of native fullscreen api 2013-02-28 15:34:44 -05:00
thatcher
d6cdd4028c removing unneccessary if 2013-02-28 15:34:43 -05:00
thatcher
a18460a310 reverting a couple changes to core openseadragon options unrelated to issue #3 . will explore these in #4 were they have already been referenced. 2013-02-28 15:34:43 -05:00
thatcher
d4b02e1aba managed to get a more complete, though hacky, implementation of #3. the big sticky point was how firefox and safari beahved when switching between applications when already in full screen mode. because we didnt have an event listener for fullscreenchange, and because those browsers released full screen on application or window change (think alt+tab or cmd+tab), you would come back to a empty document. more work left here to make this worth merging into master 2013-02-28 15:34:43 -05:00
thatcher
f602a682f7 basic implementation of enhancement request from issue #3 2013-02-28 15:34:43 -05:00
thatcher
2f938766b3 Merge branch 'master' of https://github.com/openseadragon/openseadragon 2013-02-28 14:38:12 -05:00
thatcher
015e94efa4 found typo in source, thie was supposed to be this 2013-02-28 14:37:32 -05:00
thatcher
361052656c patch for pinch zoom point. initial two point touch midpoint is used for duration of pinch-zoom operation. references #17 2013-02-27 06:57:06 -05:00
thatcher
2372968518 refs #4 - have noticed navigator is often blurry when rendered. changing blendTime can alleviate or worsen this issue if it is more or less than the animationTime (respectively). Im setting this to the value equal to animationTime as a hot patch while we figure out why these two variables are coupled. 2013-02-26 23:52:23 -05:00
Hal Blackburn
3eac183d8c Fix handling of touchmove events on Android
Dragging the canvas did not work on Android devices as the === check for
lastTouch and the current event was always evaluating to false.
Presumably Safari on iOS re-uses the same Touch object for touchmove
events with the same finger, whereas Chrome/Firefox on Android creates
new Touch objects for each event (so the === evaluates false). The code
now compares Touch.identifier to ensure the new touch event is from the
same finger as the initiating touchstart.
2013-02-26 11:50:11 +00:00
thatcher
070a48f6c8 fixing jsdocs that caused recursion 2013-02-19 22:53:54 -05:00
Ian Gilman
9569ff1215 Removed JS Hint globals comment, since it's now in grunt 2013-02-14 12:04:47 -08:00
Ian Gilman
4e51b642ef Now properly setting version in build 2013-02-14 12:02:17 -08:00
thatcher
fda24f343a added dozens of event hooks based on input from macreery's branch 2013-02-13 19:44:23 -05:00
thatcher
d8a3ccf5e8 dont compute NaN for opacity because FF doesnt like that, per macreery's branch, thanks 2013-02-13 17:20:05 -05:00
thatcher
27d2898818 added removeAllHandlers per macreery's branch, finally implemented removeHandler 2013-02-13 17:11:33 -05:00
thatcher
e851de80b8 added class names to viewer canvas and container per macreery's branch 2013-02-13 16:50:23 -05:00
thatcher
685f58d663 adding support for positional arguments for minLevel and maxLevel in DziTileSource per macreery's branch 2013-02-13 16:27:17 -05:00
thatcher
0540e834b4 refactoring after lint left an error related to drawer and overlays 2013-02-13 09:05:35 -05:00
thatcher
a0fd2b3324 added inital support for moving to a nodejs build with grunt. build only includes concant and lint. fixed all lint warning where appropriate and added lint inline ignores where appropriate. when build script is complete we will start transition to new repo. also currently researching how to prune unfriendly portion of history that make repo large due to large binary files in web app. I like git well enough but using branches for web apps unrelated to code base has made me weary once again of social web fads that force convention without long term consideration of cost. 2013-02-12 22:40:08 -05:00
thatcher
855e753225 some overlays work that had be be completed before finishing tile layers. added brief example of how to how host tiles of zoom.it 2013-02-10 22:53:51 -05:00
thatcher
aecf576c06 finished initial development of iiif info xml and json support, added example to site 2013-02-09 15:58:43 -05:00
thatcher
3f20018c75 correction for recent change which computed a highestLevel for drawer with max instead of min operation 2013-02-09 12:21:51 -05:00
thatcher
d2d5828d8f adding support for several new tile sources including iiif, osm and tms (osm and tms are thanks to seajax-utils project) 2013-02-08 15:45:22 -05:00
thatcher
576c00b37d adding support for several new tile sources including iiif, osm and tms (osm and tms are thanks to seajax-utils project) 2013-02-08 09:21:28 -05:00
thatcher
792165fdf9 accidently left alwaysBlend global defaul set to true during development. Looking forward to our new org based repo where I'll be constrained more than my current wild-west repo, sorry. corrections also to pan constrainment details which came to the forefront while working on collections zoom and panconstraints. Added new ui feature page to demo basics. 2013-02-05 21:26:40 -05:00
thatcher
b1cdaaa845 found an IE specific error in some styles set for collections visualizations 2013-02-01 15:35:46 -05:00
thatcher
808354dd6f Merge branch 'master' of github.com:thatcher/openseadragon 2013-02-01 15:19:05 -05:00
thatcher
f863f5fcae changing drawHTML routine to avoid ie rendering issue 2013-02-01 15:18:53 -05:00
ygra
354c0f8d57 Fixed comment typos 2013-01-31 19:18:32 +01:00
thatcher
bd135904b6 Merge branch 'master' of github.com:thatcher/openseadragon 2013-01-31 12:30:36 -05:00