diff options
| author | Carl Hetherington <cth@carlh.net> | 2016-05-29 00:42:55 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2016-07-01 01:05:06 +0100 |
| commit | 2894023433c3addfb04ba7ae0be534e251b641b9 (patch) | |
| tree | 6b1c5c5b7f4ae96ae3b97e044c72ab9cbe2d4c39 /src/lib/subtitle_decoder.cc | |
| parent | 079f89cdd0e7b926716237e9c76fb4a51013208e (diff) | |
First go at making the line spacing do something.
Diffstat (limited to 'src/lib/subtitle_decoder.cc')
| -rw-r--r-- | src/lib/subtitle_decoder.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/lib/subtitle_decoder.cc b/src/lib/subtitle_decoder.cc index 9fb66b74f..ef9d8384b 100644 --- a/src/lib/subtitle_decoder.cc +++ b/src/lib/subtitle_decoder.cc @@ -165,10 +165,12 @@ SubtitleDecoder::give_text (ContentTimePeriod period, sub::Subtitle const & subt dcp::VAlign v_align; if (needs_placement) { DCPOMATIC_ASSERT (i.vertical_position.line); - /* This 0.053 is an arbitrary value to lift the bottom sub off the bottom + /* This 1.015 is an arbitrary value to lift the bottom sub off the bottom of the screen a bit to a pleasing degree. */ - v_position = 1.015 - (1 + bottom_line.get() - i.vertical_position.line.get()) * 1.2 * j.font_size.proportional (72 * 11); + v_position = 1.015 - (1 + bottom_line.get() - i.vertical_position.line.get()) + * 1.2 * content()->line_spacing() * j.font_size.proportional (72 * 11); + v_align = dcp::VALIGN_TOP; } else { DCPOMATIC_ASSERT (i.vertical_position.proportional); |
