I was copying Quit<div><br></div><div>Here's the update<br><br><div class="gmail_quote">On Mon, Jan 23, 2012 at 5:43 AM, Alex Merry <span dir="ltr"><<a href="mailto:kde@randomguy3.me.uk">kde@randomguy3.me.uk</a>></span> wrote:<br> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 20/01/12 20:36, mirsal wrote:<br> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Hello Kevin,<br> <br> Thanks for the updated patch !<br> I am mostly fine with it as it is, except for that sentence:<br> <br> On Thu, 2012-01-19 at 14:22 -0500, Kevin Anthony wrote:<br> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> If the media player does not have a UI, this should not be implemented<br> </blockquote> <br> I'd say, either use "may not" or remove the sentence altogether.<br> (Don't bother sending an updated patch, We will amend it before pushing)<br> <br> @Alex: Any comment ?<br> </blockquote> <br> 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).<br> <br> Alex<br> ______________________________<u></u>_________________<br> MPRIS mailing list<br> <a href="mailto:MPRIS@lists.freedesktop.org" target="_blank">MPRIS@lists.freedesktop.org</a><br> <a href="http://lists.freedesktop.org/mailman/listinfo/mpris" target="_blank">http://lists.freedesktop.org/<u></u>mailman/listinfo/mpris</a><br> </blockquote></div><br><br clear="all"><div><br></div>-- <br>Thanks<br>Kevin Anthony<br><a href="http://www.NoSideRacing.com" target="_blank">www.NoSideRacing.com</a><div><br><div>Do you use Banshee?</div></div><div>Download the Community Extensions:</div> <div><a href="http://banshee.fm/download/extensions/" target="_blank">http://banshee.fm/download/extensions/</a></div><br> </div>