Tweak SubRip subtitle default position.

diff --git a/ChangeLog b/ChangeLog
index cfa5de09376200f09c61dc2f862f96812b371fca..87edbddc40042cd4c439a8d90edecb565d6a03d3 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2015-11-20  Carl Hetherington  <cth@carlh.net>
+
+       * Lower default position of SubRip subtitles a bit.
+
 2015-11-19  Carl Hetherington  <cth@carlh.net>
 
        * Adjust preview subtitle position to better match reality

diff --git a/src/lib/subrip_decoder.cc b/src/lib/subrip_decoder.cc
index a9065994b8cdf2dfce66dcbb4b682aebe7adf5f7..2f8b8a5f36f4bccaa64543861cd58881f9c47421 100644 (file)
--- a/src/lib/subrip_decoder.cc
+++ b/src/lib/subrip_decoder.cc
@@ -87,10 +87,10 @@ SubRipDecoder::pass (PassReason, bool)
                                        dcp::Time (_subtitles[_next].to.all_as_seconds(), 1000),
                                        0,
                                        dcp::HALIGN_CENTER,
-                                       /* This 0.95 is an arbitrary value to lift the bottom sub off the bottom
-                                          of the screen a bit.
+                                       /* This 1.015 is an arbitrary value to lift the bottom sub off the bottom
+                                          of the screen a bit to a pleasing degree.
                                        */
-                                       0.95 - ((1 + highest - i.vertical_position.line.get()) * 1.5 / 22),
+                                       1.015 - ((1 + highest - i.vertical_position.line.get()) * 1.5 / 22),
                                        dcp::VALIGN_TOP,
                                        j.text,
                                        content->outline() ? dcp::BORDER : dcp::NONE,