-
Notifications
You must be signed in to change notification settings - Fork 16
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
caMicroscope rectangle mouse event for Alter key: change from click to mouse down #40
Comments
@bridge2014 Why don't you make the fix and make a pull request? |
I changed mouse event from click to mouse down. Need help! |
Ok the issue was: we were using openseadragon-2.0 instead of openseadragon-1.0. I've reverted back to osd-1.0 and it works fine now. I've made this fix in a temprorary branch: https://github.com/camicroscope/ViewerDockerContainer/tree/markdown_mousedown_fix To get this fix to work we'll need the latest data container not the 0.9 @bridge2014: Can you please test this branch(https://github.com/camicroscope/ViewerDockerContainer/tree/markdown_mousedown_fix ) on 177? Once it's tested we can close this issue and upgrade distro to use data container 0.9.1 and the new viewer code. |
how to fix: From geoJSONHandler.js file, change code from the ALT key _right mouse click will launch featureScape button. |
reverted back to openseadragon-1.0 enable mousedown function , but this version of openseadragon can cause other problem, such as the filter function requires 2.0 or latest openseadragon library. One of error caught in browser console is: I suggest keeping openseadragon 2.0 library and use alt+ right mouse click solution. |
We're using openseadragon1.0 at all the stonybrook instances. The brightness error is due to the filtering feature which we're not releasing in this version. I'll go ahead an disable it completely. As for the right click, lets check it out on the call and decide if we want to go ahead with it. |
mouse click cause image zoom in, which is not desirable
The text was updated successfully, but these errors were encountered: