[MPlayer-dev-eng] [PATCH] stay-on-top

[Joey Parrish]

On Mon, Nov 17, 2003 at 05:27:09PM +0100, Diego Biurrun wrote:
> directx used to have an ontop suboption, I think you should remove it in
> favor of -ontop.

The patch I attached does exactly that.

> Please start sentences on a new line, groff needs that.
> 
> I'd suggest the following wording:
> 
> toggle stay-on-top.
> Supported by all video out drivers which use X11, except SDL, as well
> as directx and gl2 under Windows.
> 
> I dislike usin Win32 in the user docs, it should be Windows IMHO.

Okay.  Can do.

> > +.B \-ontop 
> > +Makes the player window stay on top other windows. Can also be toggled with
> > +the 'T' key. Supported by drivers which use X11, except SDL. Also supported
> > +by directx and gl2 under win32.
> 
> Same here.  I'd also say you don't need to mention the T key.

Yeah, that's leftover from the patch I was basing mine on.
I'll remove it.

> Is it so hard to add this to SDL as well?  Atmos, what do you say?

I won't mess with SDL.  I think a revised version can be committed
without SDL, and SDL can be added later when someone qualified gets
around to doing it.

I'll send a new patch with these DOCS revisions once someone can
tell me that the vo's that should work, do work.

--Joey

-- 
All philosophy is naive.