Page MenuHomePhabricator

Progress bar time taken is incorrect format
Open, In Progress, Needs TriagePublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  • Edit any video
  • Check on the progress bar time

What happens?:
The Time shown exceeds the 60 second mark, and we see time like 19.80s (where as it should be 20.20s).

What should have happened instead?:
The millisecond timer goes up to 100, and should go till 59.

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):
Current prod (VideoCutTool) [https://videocuttool.wmcloud.org/]

Other information (browser name/version, screenshots, etc.):
PFA screenshots

Screenshot 2025-02-27 at 21.43.06.png (746×2 px, 70 KB)

Screenshot 2025-02-27 at 21.43.05.png (746×2 px, 69 KB)

Exit criteria

  • Time should follow the basic validation.
  • Make a util function for the same.
  • Unit tests around the same util function.

Event Timeline

Thank you for tagging this task with good first task for Wikimedia newcomers!

Newcomers often may not be aware of things that may seem obvious to seasoned contributors, so please take a moment to reflect on how this task might look to somebody who has never contributed to Wikimedia projects.

A good first task is a self-contained, non-controversial task with a clear approach. It should be well-described with pointers to help a completely new contributor, for example it should clearly pointed to the codebase URL and provide clear steps to help a contributor get setup for success. We've included some guidelines at https://phabricator.wikimedia.org/tag/good_first_task/ !

Thank you for helping us drive new contributions to our projects <3

As of now we're displaying the time as received from the server end itself. Implement a basic utility function on client end, to format the time and call it while displaying it to ProgressBar

Also do write a unit test for the same utility function

Punith.nyk added a subscriber: ChiomaVero.

Since there was no response from the assignee, removing for now so that others can take it up.

Hi! I am new to the community and would love to pick this up.

hey @Marobinette removing you from the assignee here, as no updates on you regarding the ticket
feel free to re-claim it, once you start any work around this

Charlespettis changed the task status from Open to In Progress.Nov 19 2025, 9:46 PM
Charlespettis claimed this task.
Charlespettis moved this task from Incoming to Doing on the VideoCutTool board.

Hi I did this just to cut my teeth on something for this project. Sorry if I didn't follow correct process, but I need my gitlab approved before I can push and create a PR

EDIT: Merge request created!

Hi I did this just to cut my teeth on something for this project. Sorry if I didn't follow correct process, but I need my gitlab approved before I can push and create a PR

EDIT: Merge request created!

Cool.

PS: you can always fork the repo and push MRs