[MPRIS] MPRIS Fullscreen Extensions

[Kevin Anthony]

That makes sense, then you can do something akin to Fullscreen =
!Fullscreen to toggle
Here's the current patch, but i'm not sure what i should do when you said
"We also can't really say that the media player should implement these
additions, since we want to be backwards compatible"

On Mon, Jan 16, 2012 at 5:30 AM, Alex Merry <kde at randomguy3.me.uk> wrote:

>  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> 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
>> http://lists.freedesktop.org/mailman/listinfo/mpris
>>
>
>
>
>  --
> Thanks
> Kevin Anthony
> www.NoSideRacing.com
>
> Do you use Banshee?
>  Download the Community Extensions:
> http://banshee.fm/download/extensions/
>
>
>
> _______________________________________________
> MPRIS mailing listMPRIS at lists.freedesktop.orghttp://lists.freedesktop.org/mailman/listinfo/mpris
>
>
>
> _______________________________________________
> MPRIS mailing list
> MPRIS at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mpris
>
>


-- 
Thanks
Kevin Anthony
www.NoSideRacing.com

Do you use Banshee?
Download the Community Extensions:
http://banshee.fm/download/extensions/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/mpris/attachments/20120118/b1899786/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Added-Fullscreen-property.patch
Type: text/x-patch
Size: 2430 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mpris/attachments/20120118/b1899786/attachment.bin>