[MPlayer-dev-eng] ?[patch]? Bottom aligned subtitles

D Richard Felker III dalias at aerifal.cx
Sun Sep 15 04:55:01 CEST 2002


On Sat, Sep 14, 2002 at 09:17:51PM +0200, Oskar Liljeblad wrote:
> On Saturday, September 14, 2002 at 10:09, Ji?í Svoboda wrote:
> [..]
> > > > Looks good, but wouldn't it be better with a boolean toggle
> > > > for top/bottom alignment instead?
> > > 
> > > Yes much better. Then you can control it interactively with the
> > > existing subpos controls.
> > > 
> > I mainly agree, but if You have both controls You can set area of
> > screen where to put subtitles. I mean subtitles will be _never_ under
> > sub_end_pos (usefull if TV crops image) and will prefereably start at
> > sub_pos (if there is enough space between sub_pos and sub_end_pos
> > otherwise will start above sub_pos just to end at sub_end_pos) - just
> > bellow the movie.
> 
> What would happen if you set sub_end_pos to a value less than
> sub_pos? And what if you set sub_end_pos to sub_pos+1? Would there
> be any cropping done? Also, with sub_end_pos, you'd need new bindings
> for subtitle movement up/down (besides R and T).
> 
> I think it's all just better to use a boolean value sub_align_top.
> Changing this value would require the subs to be redrawn. It should
> be false by default. (I bet 95% of the users use subtitles in the
> bottom of the screen, and then bottom alignment is most natural.)
> 
> Or another approach: skip sub_align_top, use only sub_pos. If
> sub_pos is below 50 (i.e. top half of screen), use top alignment,
> otherwise bottom alignment. (Actually I outlined the code for this
> in an earlier post.)

Huh? This is total nonsense. Whether you want top/bottom alignment has
nothing to do with whether the subs are above or below 50%. It's more
related to whether you're putting them in a black border below the
movie, or else in front of the picture. In any case, there's a much
better way to do it without adding a new option: just make negative
subpos indicate bottom alignment.

Rich





More information about the MPlayer-dev-eng mailing list