From: Carl Hetherington Date: Fri, 20 Nov 2015 00:03:14 +0000 (+0000) Subject: Adjust subtitle position when referenced from top. X-Git-Tag: v2.5.7~6 X-Git-Url: https://main.carlh.net/gitweb/?p=dcpomatic.git;a=commitdiff_plain;h=e9edb9ffd2deee65b2ab044a4a8bc149ed44a0b4 Adjust subtitle position when referenced from top. SMPTE says "top" means that vposition is from the top of the screen to the top of the subtitle, but Dolby servers apparently make it top of screen to bottom of subtitle; adjust the preview to do the latter. --- diff --git a/ChangeLog b/ChangeLog index 8ed733c43..cfa5de093 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2015-11-19 Carl Hetherington + + * Adjust preview subtitle position to better match reality + in some cases. + 2015-11-19 c.hetherington * Add menu option to make a DKDM for DCP-o-matic (#755). diff --git a/src/lib/render_subtitles.cc b/src/lib/render_subtitles.cc index 9b5f585e5..26f41c89e 100644 --- a/src/lib/render_subtitles.cc +++ b/src/lib/render_subtitles.cc @@ -261,8 +261,13 @@ render_line (list subtitles, list > fonts, int y = 0; switch (subtitles.front().v_align ()) { case dcp::VALIGN_TOP: - /* v_position is distance between top of frame and top of subtitle */ - y = subtitles.front().v_position() * target.height; + /* SMPTE says that v_position is the distance between top + of frame and top of subtitle, but this doesn't always seem to be + the case in practice; Gunnar Ásgeirsson's Dolby server appears + to put VALIGN_TOP subs with v_position as the distance between top + of frame and bottom of subtitle. + */ + y = subtitles.front().v_position() * target.height - layout_height / PANGO_SCALE; break; case dcp::VALIGN_CENTER: /* v_position is distance between centre of frame and centre of subtitle */