-
Notifications
You must be signed in to change notification settings - Fork 14
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
Mirador shows no image after using of the mirador-image-tools #38
Comments
Hi, I encountered the same problem.
Is there any progress on this issue ? Thank you |
Thank you for posting this issue! We were able to recreate this issue and started investigating at community call on 10/28. Notes:
|
Hey there, |
Hi, the problem is still there. Unsupported rotation: NaN [index.html:13441:25]<...> index.html)
overrideMethod <...>/index.html:13441
onUpdateViewport <...>/main.js:85
getHandler <...>/2.main.js:6
raiseEvent <...>/2.main.js:6
(Async: FrameRequestCallback)
/*...*/ Best |
I fiddled around with this a while longer and tried to reproduce the bug in mirador dev setup to look at it a little closer, i.e. having setup of the mirador repo developing it with webpack serve (using the start scripts) and using the plugin as a preconfigured one there. In this setup the problem does not arise. I don't know what this means exactly, but made me think of some build/dependency issues as the source of the bug. Best |
I had some time again to look into it. Concerning the last comment testing a little bit longer showed, problems in the other setup are the same. Yet with the help of react dev tools in the browser I saw that the viewerConfig seems to be (part) of the problem. Leafing through a manifest in the viewer there are several points, when after changing the page the viewerConfig Prop of the OSD-viewer element and the image tools element becomes If in such a situation we change a setting in the image tools the viewerConfig gets set with only the image tool config part and breaks the viewer, now leafing forward or backward, doesn't change the config leaving it broken. |
For reproducing - use the mirador-image-tools.netlify.app instance.
The text was updated successfully, but these errors were encountered: