Image manipulation tools integration with release 2.1#989
Closed
thehabes wants to merge 10 commits intoProjectMirador:release2.1from
thehabes:thehabes-image-manipulation-tools
Closed
Image manipulation tools integration with release 2.1#989thehabes wants to merge 10 commits intoProjectMirador:release2.1from thehabes:thehabes-image-manipulation-tools
thehabes wants to merge 10 commits intoProjectMirador:release2.1from
thehabes:thehabes-image-manipulation-tools
Conversation
added 10 commits
June 15, 2016 10:01
sorry about that, I completely missed this mistake
…is is only the image manipulation html in the HUD, the event actions in imageView and the necessary CSS to go along with it. Tested with working manifests from the projectmirador.org demo (success). Also worked with non-IIIF images, but this commit does not include non-IIIF image support, that will be next.
These are the image manipulations tools html in the hud, their event handlers and the CSS to the support them.
Author
|
Meh got it wrong, had the range code too! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I think I did this correctly with the branching this time.
I did not write tests or anything, but I wanted to get this up since others working with it. I am not sure how to write tests for it, I will need your help with that. It's definitely not perfect, but its as worked in as I could get it. For a running mock up of the UI/UX, see http://165.134.241.141/brokenBooks/getManifest.html.