Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#5334 closed Feature Request (fixed)

Sub-second accurate time display

Reported by: aseguralasa Owned by: kasper93
Priority: normal Milestone: 1.7.9
Component: User Interface Version: 1.7.8
Severity: normal Keywords:
Cc: Evaluation:

Description

The time indicator only displays seconds, but that is not always enough. Especially when advancing frame-by-frame one often wants to know the exact position in the video.

The time display could (optionally) show either "frame number" within the second, or "hundreds of a second" (cs) or "milliseconds" after the value in seconds.

So, when the "Accurate time" option is enabled we would see e.g.:

13:45.20     -> if using cs or frame number, or
13:45.200    -> if using ms

I'm not sure now what the typical unit for this is.

Attachments (1)

5334.png (11.0 KB) - added by kasper93 5 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 5 years ago by Underground78

Cc: Underground78 added
Resolution: support
Status: newclosed

You can achieve what you want by opening the Subresync panel which can be used to sync subtitles and will also engage the "high precision" timer.

comment:2 Changed 5 years ago by aseguralasa

Sorry, but that is not a complete solution.

The Subresync panel is only available when there are subtitles, otherwise the option is disabled.

And even with subtitles that option opens a panel with the complete list of subtitle strings. And yes, it also enables sub-second time precision.

My request was for any video (even without subtitles or without audio).

comment:3 Changed 5 years ago by Underground78

Resolution: support
Status: closedreopened

It seems I misread your ticket, I thought you wanted to use higher precision to sync your subtitles.

comment:4 Changed 5 years ago by kasper93

@underground78: We had similar requests before, maybe advanced option would be so bad?

comment:5 Changed 5 years ago by Underground78

I fear that if we use an advanced option users will find it too unpractical.

I can do that if you think it would actually make people happy. Another solution would be to automatically engage the high precision mode when doing frame stepping but again I'm not sure if it's really what most users want.

comment:6 Changed 5 years ago by kasper93

I fear that if we use an advanced option users will find it too unpractical.

I see what you mean. How about simple context menu then? Take a look at attachment.

EDIT: Patch https://github.com/kasper93/mpc-hc/commit/4d4f1189287813d4370b3f5185a78b8c786e2a8b

Last edited 5 years ago by kasper93 (previous) (diff)

Changed 5 years ago by kasper93

Attachment: 5334.png added

comment:7 Changed 5 years ago by Underground78

I don't have any real opinion, fine I guess, not sure if it's overkill or not.

comment:8 Changed 5 years ago by Underground78

Message from aseguralasa that got blocked by the anti-spam:

I would make this optional as for most uses milliseconds might be clutter.

A toggle option would be under Tweaks, Miscelaneous or Advanced.

comment:9 Changed 5 years ago by kasper93

Owner: set to kasper93
Resolution: fixed
Status: reopenedclosed

In 141149:

CPlayerStatusBar: Add context menu with timer options.

Allow to enable high precision timer also when subresync bar is hidden.

Fixes #5334

comment:10 Changed 5 years ago by kasper93

Cc: Underground78 removed
Milestone: next release

Will be in the nightly builds as of version 1.7.8.150.

comment:11 in reply to:  10 Changed 5 years ago by ever_green

Replying to kasper93:

Will be in the nightly builds as of version 1.7.8.150.

Why did you omit the word "timer"? I think "High precision timer" is better than "High precision". "High precision" becomes a litle strange when it is translated into Japanese.

Last edited 5 years ago by ever_green (previous) (diff)

comment:12 Changed 5 years ago by thevbm

Milestone: next release1.7.9

Milestone renamed

Note: See TracTickets for help on using tickets.