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

Regression A/V problem: multiple media files in one canvas #779

Open
1 of 3 tasks
mcwhitaker opened this issue Feb 5, 2021 · 12 comments
Open
1 of 3 tasks

Regression A/V problem: multiple media files in one canvas #779

mcwhitaker opened this issue Feb 5, 2021 · 12 comments

Comments

@mcwhitaker
Copy link

mcwhitaker commented Feb 5, 2021

UV version:

https://uv-v3.netlify.app/

I'm submitting a:

Current behavior:
We added a link to the UV in the IIIF Cookbook for recipe # 64, Opera in One Canvas. This recipe demonstrates the ability to use one single canvas for multiple A/V files, each using a fragment of the time. Part of approving the process was to demo it, which was done using UV (see recording on minute 38:58).

The link demonstrated in the video no longer works. UV does not play the second file.

Expected behavior:
We expected the UV link in the recipe mentioned above to still work.

Steps to reproduce:
Go to the recipe, and click on the link View in Universal Viewer and try to navigate to Atto Secondo.

Related code:

// insert any relevant code here

Other information:

@edsilv edsilv added this to To do in v4 Mar 3, 2021
@glenrobson
Copy link

Has this been fixed? I was just testing the cookbook UV link and the table of contents seems to be working :-).

@mcwhitaker
Copy link
Author

Yes, but now the video doesn't show on the viewer? The playhead moves and is correctly showing two files. @glenrobson is the video playing correctly for you?

@glenrobson
Copy link

Ah no... you're right I got too carried away with the playhead moving and didn't click play...

@edsilv edsilv added this to todo in AV Component Jun 11, 2021
@edsilv edsilv removed this from To do in v4 Sep 13, 2021
@edsilv edsilv added this to todo in v3 Sep 13, 2021
@edsilv edsilv removed this from todo in AV Component Sep 13, 2021
@edsilv edsilv moved this from todo to in progress in v3 Sep 29, 2021
@edsilv edsilv moved this from in progress to done in v3 Oct 1, 2021
edsilv added a commit that referenced this issue Oct 1, 2021
@markpbaggett
Copy link

Hi @edsilv, the commit above suggests that this issue was fixed, but it still seems to be failing in v3 and v4 of UV. Is this a known issue, or something that has popped back up?

@edsilv
Copy link
Member

edsilv commented Jun 28, 2022

@markpbaggett This appears to be working in 4.0.1? https://www.universalviewer.dev/#?c=&m=&cv=&xywh=-2494%2C-197%2C7556%2C3936&rid=%5Bobject%20Object%5D&t=11.57800006866455

However, @stephenwf there's a scaling issue with the video where it's overlapping the share and download buttons.

@stephenwf
Copy link
Contributor

Looks like that might be unrelated to video. Fixed inline by removing the style="height: 570px" which doesn't take into account the 20px of padding in the parent on 2 inner elements (.views and .resourcesView > ul)

Screenshot 2022-06-28 at 15 28 54

@markpbaggett
Copy link

@edsilv and @stephenwf, hmmmm. I'm still getting the same error in v4.0.1 with the manifest in the recipe:

https://iiif.io/api/cookbook/recipe/0064-opera-one-canvas/manifest.json

Everything is good until we get to the second annotation on the Canvas at 3971.24. The duration keeps going in the viewer, but the second annotation doesn't appear.

I also threw together a quick gist of my own that has the same problem:

https://gist.githubusercontent.com/markpbaggett/f742e2bc857d2e76de8fdcbeb002c3fc/raw/8ac894955c43ce29c26abe2e3241d997f9faab94/wwiioh_2002.json

All is good until we get to the second annotation.

@LlGC-szw
Copy link

All issues will be triaged for further investigation or closure by the 28 September 2023. If your issue is still relevant and would like for it be investigated further please comment by 14 September 2023.

@edsilv edsilv moved this from Todo to In progress in Community Sprint #5 Nov 13, 2023
@edsilv edsilv moved this from In progress to Todo in Community Sprint #5 Nov 28, 2023
@edsilv edsilv moved this from Todo to Won't do (this time) in Community Sprint #5 Nov 29, 2023
@edsilv
Copy link
Member

edsilv commented Jan 4, 2024

@markpbaggett
Copy link

@edsilv, I'm not sure if it's related or not, but maybe. It's hard for me to tell what that Figgy fix was doing.

Some other things I noticed: UV seems to be fetching the painting annotation on the second canvas just fine. It shows up on the network tab of a page with the viewer but it doesn't paint it in UV.

Apologies for not providing better feedback or responding sooner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Community Sprint #5
Won't do (this time)
v3
review
Development

No branches or pull requests

7 participants