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

Staging #759

Merged
merged 19 commits into from
Feb 12, 2025
Merged

Staging #759

merged 19 commits into from
Feb 12, 2025

Conversation

arnayv-47
Copy link
Contributor

Changes approved by PO

Jira Story: https://cm-jira.usa.gov/browse/OGPWEB-16803

Preview Link: https://federalist-4be496a3-cca3-4187-b4d4-c3d573ae3139.sites.pages.cloud.gov/preview/gsa/ussm/staging/m3/

Add new videos and establish m3 video section. Add js player to allow videos to play in section without having to reload page. Also updated js to allow for title to change below video player.

Use: The main video player is updated whenever a thumbnail is clicked. Once a thumbnail on the right column is clicked the title under the main video will update and along with the video. This allows all three videos to be played on the same screen without having to reload the page as it's handled internally in the m3.js.

@arnayv-47 arnayv-47 requested review from km719 and attorri February 12, 2025 14:18
@arnayv-47 arnayv-47 self-assigned this Feb 12, 2025
if (!newSrc.includes("autoplay=1")) {
newSrc = newSrc.includes("?") ? `${newSrc}&autoplay=1` : `${newSrc}?autoplay=1`;
}
mainVideo.setAttribute("src", newSrc);

Check warning

Code scanning / CodeQL

DOM text reinterpreted as HTML Medium

DOM text
is reinterpreted as HTML without escaping meta-characters.
@arnayv-47 arnayv-47 merged commit 8340e58 into master Feb 12, 2025
5 checks passed
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.

3 participants