Fix default placement of SRT (etc.) now that the alignment has changed
authorCarl Hetherington <cth@carlh.net>
Mon, 19 Sep 2022 21:08:21 +0000 (23:08 +0200)
committerCarl Hetherington <cth@carlh.net>
Mon, 19 Sep 2022 21:08:21 +0000 (23:08 +0200)
src/lib/text_decoder.cc

index 930262a74da321fa22ac2df75bd2127c46592608..c691f795d2d5133f7b64e63a44275a72ce0abf30 100644 (file)
@@ -167,10 +167,10 @@ TextDecoder::emit_plain_start (ContentTime from, sub::Subtitle const & sub_subti
                                switch (line.vertical_position.reference.get_value_or(sub::BOTTOM_OF_SCREEN)) {
                                case sub::BOTTOM_OF_SCREEN:
                                case sub::TOP_OF_SUBTITLE:
                                switch (line.vertical_position.reference.get_value_or(sub::BOTTOM_OF_SCREEN)) {
                                case sub::BOTTOM_OF_SCREEN:
                                case sub::TOP_OF_SUBTITLE:
-                                       /* This 1.015 is an arbitrary value to lift the bottom sub off the bottom
+                                       /* This 0.9 is an arbitrary value to lift the bottom sub off the bottom
                                           of the screen a bit to a pleasing degree.
                                           */
                                           of the screen a bit to a pleasing degree.
                                           */
-                                       v_position = 1.015 -
+                                       v_position = 0.9 -
                                                (1 + bottom_line.get() - line.vertical_position.line.get()) * multiplier;
 
                                        v_align = dcp::VAlign::TOP;
                                                (1 + bottom_line.get() - line.vertical_position.line.get()) * multiplier;
 
                                        v_align = dcp::VAlign::TOP;