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

Release 7.7! 🎉 #5683

Merged
merged 600 commits into from
Feb 20, 2024
Merged

Release 7.7! 🎉 #5683

merged 600 commits into from
Feb 20, 2024

Conversation

cjcolvar
Copy link
Member

No description provided.

Dananji and others added 30 commits November 2, 2023 08:44
Add text-nowrap class to item page buttons
* Use active section/timespan items in Ramp to populate add to playlist options

* Add form validation, fix playlist form reset after success

* Disable/enable current track option based on the player's currentTime

* Update app/assets/javascripts/ramp_utils.js

Co-authored-by: Mason Ballengee <68433277+masaball@users.noreply.github.com>

* Update app/assets/javascripts/ramp_utils.js

Co-authored-by: Mason Ballengee <68433277+masaball@users.noreply.github.com>

* Update app/assets/javascripts/ramp_utils.js

Co-authored-by: Mason Ballengee <68433277+masaball@users.noreply.github.com>

---------

Co-authored-by: Mason Ballengee <68433277+masaball@users.noreply.github.com>
Shorten title for encodes using s3 urls
Move administrative buttons to top of page and outside of scope of Ramp
…al for screen readers

Links need to be non-navigable in order for role="presentation" or aria-hidden="true" to take effect.  I copied what I saw YouTube as doing with aria-hidden="true" and tabindex=-1.
Empty alt text for images marks them as presentation and thus skipped by screen readers.
This is easier to do than fix the underlying issue by reworking the code.  While investigating this I found that lsof is blocking for a really long time in the docker container so it is always timing out.  This makes this feature pretty useless which was another reason for this quick solution.
Mark thumbnails and wrapping links in search results are presentational for screen readers
Add canvas reference to all root ranges
Add functionality to expand/close all sections button on item page
The solr_doc includes the master file ids in the other identifier field.
These ids are not included in the accessor on the media object. These
ids also are of a different format than the media object's normal
other_identifiers. We need to exclude the master file ids from the
accessor on the proxy because of those factors.
Move add to playlist button and add icon in front of button text
masaball and others added 20 commits February 14, 2024 11:30
Use released versions of dependencies
…ilation due to uglifier not supporting newer versions of ECMAScript
Fix for asset precompilation in docker build
Install gems needed for reindex script
Add rake task to migrate IndexedFile captions to ActiveStorage
Bump version number to prepare for release
@cjcolvar cjcolvar mentioned this pull request Feb 19, 2024
32 tasks
masaball and others added 6 commits February 20, 2024 10:38
Captions are currently contained in the HLS manifest and these are
parsed by VideoJS. We do not need to manually provide text tracks to the
embedded player because of this. This commit also includes a change to
the filtering of captions for the embedded player for if/when we reenable
manually providing them so that issues will not arise for implementers who
run the caption file migration.
Enable create timeline and create thumbnail buttons on timeout
Turn off feeding caption tracks into embedded player
Copy link
Contributor

@joncameron joncameron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎊

@cjcolvar cjcolvar merged commit eb51ca5 into main Feb 20, 2024
3 checks passed
@joncameron joncameron mentioned this pull request Mar 22, 2024
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants