This repository has been archived by the owner. It is now read-only.

Add fill end times option to chapter editor #1887

Closed
ndjamena opened this Issue Feb 12, 2017 · 1 comment

Comments

2 participants
@ndjamena

ndjamena commented Feb 12, 2017

Add an option to the chapter editor to fill in the "end timecodes" fields of each of the chapters with the start time of the next chapter (chronologically speaking).

If the chapters aren't attached to an MKV the option could be disabled or the last chapters end time could simply be left blank.

Some people fill them in because they think it's a good idea, some people add them because they mistakenly think they're supposed to. The option would also help when end times are already included in the chapters and a user is repositioning the chapters. Otherwise they'd need to endlessly copy/paste the start times into the end times to keep them up to date.

When I was attempting to write a chapter editor it automatically set the end times to the start time of the next chapter when "ordered chapters" was disabled. A similar option in the MKVToolnix GUI might be useful but not required. It also showed chapter duration.

An option to wipe the end times could also be a possibility.

I don't know if it's possible but adding the option for some visual clue on the "start time" field when its timecode occurs before the next physical chapter might be useful in some situations. (I'm aware Matroska chapters don't need to be in order.)

mbunkus added a commit that referenced this issue Jul 14, 2017

GUI: chapters: add "calculate & set end timestamps" functionality
The function is available from the "additional modifications" dialog.

For most entries the smallest start timestamp of all chapters on the
same level higher than the current chapter's start timestamp will be
used as its end timestamp.  If there is no such chapter, the parent
chapter's end timestamp will be used instead.

If the chapters were loaded from a Matroska file, the end timestamp
for very last chapter on the top-most level will be derived from the
file's duration.

Implements #1887.

@mbunkus mbunkus closed this Jul 14, 2017

@mbunkus

This comment has been minimized.

Show comment
Hide comment
@mbunkus

mbunkus Jul 15, 2017

Owner

New pre-builds for Windows that include this are available (build numbers 01527 and higher).

Owner

mbunkus commented Jul 15, 2017

New pre-builds for Windows that include this are available (build numbers 01527 and higher).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.