[MPRIS] MPRIS Fullscreen Extensions

[Kevin Anthony]

I was copying Quit

Here's the update

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

> On 20/01/12 20:36, mirsal wrote:
>
>> Hello Kevin,
>>
>> Thanks for the updated patch !
>> I am mostly fine with it as it is, except for that sentence:
>>
>> On Thu, 2012-01-19 at 14:22 -0500, Kevin Anthony wrote:
>>
>>> If the media player does not have a UI, this should not be implemented
>>>
>>
>> I'd say, either use "may not" or remove the sentence altogether.
>> (Don't bother sending an updated patch, We will amend it before pushing)
>>
>> @Alex: Any comment ?
>>
>
> I think it would be better to replace the sentence with something like
> "This property is optional. Clients should handle its absence gracefully."
> (same wording as DesktopEntry on the root interface).
>
> Alex
> ______________________________**_________________
> MPRIS mailing list
> MPRIS at lists.freedesktop.org
> http://lists.freedesktop.org/**mailman/listinfo/mpris<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/20120123/4b45cdfc/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-PATCH-Added-Fullscreen-property.patch
Type: text/x-patch
Size: 2464 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mpris/attachments/20120123/4b45cdfc/attachment.bin>