From ebcdc90fa32c22895b958126ba5a6a4d562616ad Mon Sep 17 00:00:00 2001
From: Carl Hetherington <cth@carlh.net>
Date: Sun, 5 Oct 2014 23:53:54 +0100
Subject: More sensible default size / position for .srt subs.

---
 src/lib/subrip_decoder.cc | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'src/lib/subrip_decoder.cc')

diff --git a/src/lib/subrip_decoder.cc b/src/lib/subrip_decoder.cc
index e2bdc347b..2ae285b5e 100644
--- a/src/lib/subrip_decoder.cc
+++ b/src/lib/subrip_decoder.cc
@@ -52,6 +52,8 @@ SubRipDecoder::pass ()
 	if (_next >= _subtitles.size ()) {
 		return true;
 	}
+
+	/* XXX: we are ignoring positioning specified in the file */
 	
 	list<dcp::SubtitleString> out;
 	for (list<SubRipSubtitlePiece>::const_iterator i = _subtitles[_next].pieces.begin(); i != _subtitles[_next].pieces.end(); ++i) {
@@ -60,10 +62,11 @@ SubRipDecoder::pass ()
 				"Arial",
 				i->italic,
 				dcp::Color (255, 255, 255),
-				72,
+				/* .srt files don't specify size, so this is an arbitrary value */
+				48,
 				dcp::Time (rint (_subtitles[_next].period.from.seconds() * 250)),
 				dcp::Time (rint (_subtitles[_next].period.to.seconds() * 250)),
-				0.9,
+				0.2,
 				dcp::BOTTOM,
 				i->text,
 				dcp::NONE,
-- 
cgit v1.2.3