Closed Bug 1779122 Opened 2 years ago Closed 1 year ago

Video duration: render and update

Categories

(Toolkit :: Picture-in-Picture, enhancement)

enhancement
Points:
5

Tracking

()

RESOLVED FIXED
110 Branch
Tracking Status
firefox110 --- fixed

People

(Reporter: asafko, Assigned: cmkm)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fidefe-MR1-2022])

Attachments

(5 files)

User story
As a user, I want to be able to know how long the video is and what is my position in it, so I can make quick decisions about my viewing experience.

Acceptance Criteria

  1. Video duration and my current position in it are visible alongside other controls as I hover over the PiP window.

  2. Current position in the video timeline is dynamically updated as I rewind the video and with the passage of time.

  3. Duration is removed on screen sizes under [widthxheight] - our initial acceptance criteria was 580x320, but we can see if it makes sense empirically.

  4. Video duration format is based on the combination of the current time and video duration:

a. [m:ss / m:ss] videos from 1:00 to 9:59 min long

b. [mm:ss /mm:ss] videos from 10:00 min to 59:59 min long AFTER they hit the double digits in current time
[m:ss / mm:ss] format is followed until 10 min mark is reached

c. [h:mm:ss / h:mm:ss] - videos from 1:00:00 to 9:59:59 long AFTER they hit 1-hour mark
[mm:ss / h:mm:ss] format is followed until the 1-hour mark is reached

d. [hh:mm:ss /hh:mm:ss] - videos over 10 hours AFTER they hit 10-hour mark
[h:mm:ss / hh:mm:ss] format is followed until the 10-hour mark is reached

UI specs
font for macOS: SF Pro
font for Win: Segoe UI
padding left: 24 px
font size: 14 px
color: #FFFFFF

Folder with designs

Attached image PIP3.png
Attached image under an hour.png
Attached image over an hour.png
Whiteboard: [fidefe-MR1-2022]
Summary: Render and update video duration → Video duration: render and update

Also attaching our font usage guide here

Severity: -- → N/A
Type: task → enhancement
Depends on: 1579645
Assignee: nobody → nbaumgardner
No longer depends on: 1579645
Assignee: nbaumgardner → cmeador
Status: NEW → ASSIGNED
Points: --- → 5

Added 1779119 as a blocker, as the work Niklas is doing in that patch will lay the foundation for this one.

Depends on: 1779119
Attachment #9306132 - Attachment description: WIP: Bug 1779122 - WIP → WIP: Bug 1779122 - Add PiP video timestamp
Attachment #9306132 - Attachment description: WIP: Bug 1779122 - Add PiP video timestamp → Bug 1779122 - Add PiP video timestamp r?niklas!,#pip-reviewers
Blocks: 1806734
Pushed by cmeador@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f31bb7fda86f
Add PiP video timestamp r=niklas,pip-reviewers
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
Blocks: 1808825
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: