Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Zoomify image base layer #312

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

krdyke
Copy link
Contributor

@krdyke krdyke commented Jun 5, 2014

I added this feature to address a local need (cc @jschell42), but thought it was worth seeing if you all would be interested. My rationale for its inclusion is that it would be another, fairly easy route to work with a static, non-georeferenced image, with a major advantage over static images in that you have the speed of tiling. As I'm sure you're aware, for scanned historical maps, it frequently is the case that they don't respond well to georeferencing and are best handled outside of a web map environment. However, for detailed maps, the use of a single, static image can sometimes be pretty slow.

In the exhibit form, I added options for the Zoomify base URL, as well as the image width and height, which are included in the ImageProperties.xml file for any Zoomify-ed image. See the included image.

For a working example, see http://omeka.kevinrdyke.com/neatline/show/zoomify

Thanks!

screen shot 2014-06-05 at 3 08 47 pm

@jamiefolsom jamiefolsom modified the milestone: 2.5.2 Apr 3, 2017
@jamiefolsom jamiefolsom modified the milestone: 2.5.2 Jun 21, 2017
@jamiefolsom
Copy link
Member

Hi @jeremyboggs! I believe a fix for the issue this PR addressed was made is in #433, and that that PR supersedes this one. Is that correct, @akstuhl? If so, should we contact @krdyke and close this PR? Thanks!

@akstuhl
Copy link
Collaborator

akstuhl commented Nov 21, 2017

Hi @jamiefolsom - #433 rebased and extended a different PR (also submitted by @krdyke) that has since been closed. It didn't address support for the Zoomify layer type, which is the focus of this PR. Would it make sense to open a new issue to review usefulness of Zoomify support to the main fork? If that becomes a priority, this PR could then be similarly updated to the current code base.

@jamiefolsom
Copy link
Member

Ah, right you are, @akstuhl -- my mistake! I like the idea of a zoomable base image, but I think that's likely something we won't undertake on 2.6.x, since we're aiming to introduce only fixes for now. Fine to leave this as-is, until we confirm that direction. Thanks!

@akstuhl
Copy link
Collaborator

akstuhl commented Nov 21, 2017

Agreed, thanks @jamiefolsom

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants