Page MenuHomePhabricator

Display time taken to encode the video
Closed, ResolvedPublic

Description

Display time taken to encode the video editings i.e if the video encoding took 3min 20 sec to process, display the time along with the newly generated video.

Event Timeline

Sandyabhi subscribed.

Change 701692 had a related patch set uploaded (by Sandyabhi; author: Sandyabhi):

[labs/tools/VideoCutTool@master] Add the timer

https://gerrit.wikimedia.org/r/701692

Hi @Sandyabhi, Does "Time taken : " string seems like too big than the other headings on the screen?

image.png (808×1 px, 164 KB)

@Sandyabhi Seems good to me, Please feel free to update the patch set and set for the review :)

Gopavasanth removed a project: Patch-For-Review.

@Sandyabhi as the code changed a lot, please feel free to claim again if you still want to work on this task. Thanks!

Change 701692 abandoned by Gopavasanth:

[labs/tools/VideoCutTool@master] Add the timer

Reason:

Thanks for your patch, Source code is changed a lot so please feel free to rework on the new code.

https://gerrit.wikimedia.org/r/701692

hey,@Gopavasanth @Soda .Any suggestions for improvement?

image.png (456×1 px, 138 KB)

And also, the long titles are getting shadowed out by the button should we also fix that?

Change 882690 had a related patch set uploaded (by Punith.nyk; author: Punith.nyk):

[labs/tools/VideoCutTool@master] T278626 Add and display the time taken to encode the video

https://gerrit.wikimedia.org/r/882690

Change 882690 had a related patch set uploaded (by Gopavasanth; author: Punith.nyk):

[labs/tools/VideoCutTool@master] Display time taken to encode the video

https://gerrit.wikimedia.org/r/882690

Change 882690 merged by jenkins-bot:

[labs/tools/VideoCutTool@master] Display time taken to encode the video

https://gerrit.wikimedia.org/r/882690