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

feat(remaining-time-display): displays remaining time in x-small or tiny #296

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

amtins
Copy link
Member

@amtins amtins commented Jan 27, 2025

Description

When the current time and media duration are displayed, it's easy for the user to visualize the time remaining to play. However, when the player is displayed in x-small or tiny, no time indication is shown.

Changes made

  • modify _layout.scss to display the remaining time when the player is displayed in x-small or tiny

When the current time and media duration are displayed, it's easy for the user
to visualize the time remaining to play. However, when the player is displayed
in x-small or tiny, no time indication is shown.

- modify `_layout.scss` to display the remaining time when the player is
displayed in x-small or tiny.
Copy link

PR Preview Action v1.6.0

🚀 View preview at
https://SRGSSR.github.io/pillarbox-web/pr-preview/pr-296/

Built to branch gh-pages at 2025-01-27 09:24 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Copy link

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
98.11% (-0% 🔻)
832/848
🟢 Branches
93% (-0.03% 🔻)
385/414
🟢 Functions 98.69% 226/229
🟢 Lines
98.5% (-0% 🔻)
789/801
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🟢
... / srgssr.js
99.37% (-0.01% 🔻)
98.94% (-0.02% 🔻)
100%
99.27% (-0.01% 🔻)

Test suite run success

247 tests passing in 10 suites.

Report generated by 🧪jest coverage report action from 7dedb56

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🍿 Code Review
Development

Successfully merging this pull request may close these issues.

1 participant