avalonmediasystem / avalon

Avalon Media System – Samvera Application
http://www.avalonmediasystem.org/
Apache License 2.0
94 stars 51 forks source link

Review Ramp UI #5176

Closed elynema closed 1 year ago

elynema commented 1 year ago

Description

There are a number of ways that Ramp UI differs from the current mediaelement.js player UI in Avalon. Need to review these and identify any UI changes to the Ramp UI to make it work well for current Avalon users.

Example media object in current Avalon: https://media.dlib.indiana.edu/media_objects/qn59qp839 Example record in Ramp: https://iiif-react-media-player.netlify.app/

Description of done

elynema commented 1 year ago

Goals for updated media object page design

@Dananji @joncameron Here are a list of goals for working with Matt on layout of media object page. Please edit or add more if you have any other thoughts.

Dananji commented 1 year ago

This is not relevant to this implementation since we are getting rid of the Picture-In-Picture button in Ramp, but I wanted to correct this;

Regarding the Picture-In-Picture button in Ramp, I was incorrect about the comment on the button not working for stream media. The button was disabled in Firefox because the browser provides the functionality by default, but it's enabled in Chrome because it is not on by default. So VideoJS turns it on/off based on the browser compatibility.

elynema commented 1 year ago

Summary of action items for media player buttons from 5/16/2023:

These tasks will be added to other tickets, so just documenting here for now. Major changes to add to playlist and lil scrub will likely need to be reviewed by the MCO working group, but general video.js player in Ramp will be close enough to current mediaelement.js player in Avalon with the proposed changes here that we're not worried about getting feedback on basic button functionality.

We will want to share the Ramp changes with other Ramp users: UT Austin and HistoryMakers.

Dananji commented 1 year ago

One more thing to add: when the structure has long text, Ramp doesn't render them in a user-friendly manner. I think we could probably render these long texts in a more elegant way that's easier to comprehend by the user.

This image shows how these texts are displayed when the structured navigation component spans the full width of the page. Screenshot from 2023-05-16 15-32-19

And this is how it renders when the structured navigation component doesn't span the entire width of the page, and this could make the structured navigation component very long making it more un-usable; Screenshot from 2023-05-16 15-34-44

The approach we have now in the sections in Avalon where the structure can be collapsed and expanded or something similar could be explored as a solution for this.

elynema commented 1 year ago

Note that MCO working group staff report using the add to playlist with time-based clips of an audio/video item, so we do need to retain this functionality. We may be able to integrate this into a bar of buttons just below the video player, alongside the create thumbnail button. See page re-design work on #5202.

elynema commented 1 year ago

Li'l Scrub will be handled via other issues (see Avalon #5216 for reviewing patterns in other players and Ramp #12 for creating functionality).

elynema commented 1 year ago

on 5/26/23, Emily reached out to Adam Krone (ThirdWave with History Makers) and Larry Yang (UT Austin) about upcoming changes to Ramp and pointed them to the issues in the Ramp repository. I offered the opportunity to ask questions or schedule a conversation. As of 6/29/23, I have not heard back.

Once the metadata component and supplemental files component are available in Ramp (and now that some media player UI changes have been made on the demo site), I should probably reach out to them to let them know a demo is available. Let's do that when #189 (change to audio adjustment bar), #173 (supplemental file component) are completed.

larry.yang@austin.utexas.edu larry.yang@austin.utexas.edu adam.krone@thirdwavellc.com