[MPRIS] MPRIS Fullscreen Extensions

[Alex Merry]

Nope, mailing list is fine.  I'll add a HACKING file to the tree.

I'm not sure whether the root interface is the right place for this, or 
whether it should be in Player.  I guess it fits with Raise, though.

I think Fullscreen should be a property - unlike Raise or Quit, it is a 
state (either you are fullscreen or you are not) and so requesting the 
value makes sense.  Also, Fullscreen is a noun, not a verb.

I also think (for the same reasons) that CanFullscreen should be 
CanSetFullscreen.  We also can't really say that the media player should 
implement these additions, since we want to be backwards compatible.  So 
clients will need to cope gracefully with the relevant 
properties/methods not existing.

Alex


On 16/01/12 00:24, Kevin Anthony wrote:
> I couldn't find a hacking file, so i'm unsure where to submit the 
> patch.  So i figured i'd submit it to the mailing list.  If i'm 
> supposed to submit it somewhere else, please let me know and i'll 
> submit it there.
>
> Thanks
> Kevin Anthony
>
> On Sun, Jan 15, 2012 at 5:07 PM, Alex Merry <kde at randomguy3.me.uk 
> <mailto:kde at randomguy3.me.uk>> wrote:
>
>     On 14/01/12 18:41, Kevin Anthony wrote:
>
>         I would like to propose adding two methods, or one method with
>         a boolean flag, that allows, if a video was playing, to
>         fullscreen/un-fullscreen the players.
>
>         Although i have nothing to back this up, i think most people
>         watch there movies in fullscreen most of the time.  And i feel
>         any remote would be incomplete without being able to
>         fullscreen the players.
>
>
>     Hmm... that's not a bad idea.  Where I can see it being most
>     useful is for LIRC-style controllers that use MPRIS (or
>     bluetooth-based phone remotes etc).
>
>     My suggestion would be two Player properties:
>     FullScreen: boolean, read/write
>     CanSetFullScreen: boolean, read-only
>     the second determining whether the first can be written to.
>
>     Alex
>     _______________________________________________
>     MPRIS mailing list
>     MPRIS at lists.freedesktop.org <mailto:MPRIS at lists.freedesktop.org>
>     http://lists.freedesktop.org/mailman/listinfo/mpris
>
>
>
>
> -- 
> Thanks
> Kevin Anthony
> www.NoSideRacing.com <http://www.NoSideRacing.com>
>
> Do you use Banshee?
> Download the Community Extensions:
> http://banshee.fm/download/extensions/
>
>
>
> _______________________________________________
> MPRIS mailing list
> MPRIS at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mpris

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mpris/attachments/20120116/03f5c785/attachment.htm>