<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Nope, mailing list is fine.&nbsp; I'll add a HACKING file to the tree.<br>
    <br>
    I'm not sure whether the root interface is the right place for this,
    or whether it should be in Player.&nbsp; I guess it fits with Raise,
    though.<br>
    <br>
    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.&nbsp; Also, Fullscreen is a noun, not a
    verb.<br>
    <br>
    I also think (for the same reasons) that CanFullscreen should be
    CanSetFullscreen.&nbsp; We also can't really say that the media player
    should implement these additions, since we want to be backwards
    compatible.&nbsp; So clients will need to cope gracefully with the
    relevant properties/methods not existing.<br>
    <br>
    Alex<br>
    <br>
    <br>
    On 16/01/12 00:24, Kevin Anthony wrote:
    <blockquote
cite="mid:CA+C4C6fVNwsmCGzx5-CYgxZFWMG6Bzvan_mWSbsh_k72nhF1kQ@mail.gmail.com"
      type="cite">I couldn't find a hacking file, so i'm unsure where to
      submit the patch. &nbsp;So i figured i'd submit it to the mailing list.
      &nbsp;If i'm supposed to submit it somewhere else, please let me know
      and i'll submit it there.
      <div>
        <br>
      </div>
      <div>Thanks</div>
      <div>Kevin Anthony<br>
        <br>
        <div class="gmail_quote">On Sun, Jan 15, 2012 at 5:07 PM, Alex
          Merry <span dir="ltr">&lt;<a moz-do-not-send="true"
              href="mailto:kde@randomguy3.me.uk">kde@randomguy3.me.uk</a>&gt;</span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">On
            14/01/12 18:41, Kevin Anthony wrote:<br>
            <blockquote class="gmail_quote" style="margin:0 0 0
              .8ex;border-left:1px #ccc solid;padding-left:1ex">
              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.<br>
              <br>
              Although i have nothing to back this up, i think most
              people watch there movies in fullscreen most of the time.
              &nbsp;And i feel any remote would be incomplete without being
              able to fullscreen the players.<br>
            </blockquote>
            <br>
            Hmm... that's not a bad idea. &nbsp;Where I can see it being most
            useful is for LIRC-style controllers that use MPRIS (or
            bluetooth-based phone remotes etc).<br>
            <br>
            My suggestion would be two Player properties:<br>
            FullScreen: boolean, read/write<br>
            CanSetFullScreen: boolean, read-only<br>
            the second determining whether the first can be written to.<br>
            <br>
            Alex<br>
            _______________________________________________<br>
            MPRIS mailing list<br>
            <a moz-do-not-send="true"
              href="mailto:MPRIS@lists.freedesktop.org" target="_blank">MPRIS@lists.freedesktop.org</a><br>
            <a moz-do-not-send="true"
              href="http://lists.freedesktop.org/mailman/listinfo/mpris"
              target="_blank">http://lists.freedesktop.org/mailman/listinfo/mpris</a><br>
          </blockquote>
        </div>
        <br>
        <br clear="all">
        <div><br>
        </div>
        -- <br>
        Thanks<br>
        Kevin Anthony<br>
        <a moz-do-not-send="true" 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 moz-do-not-send="true"
            href="http://banshee.fm/download/extensions/"
            target="_blank">http://banshee.fm/download/extensions/</a></div>
        <br>
      </div>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
MPRIS mailing list
<a class="moz-txt-link-abbreviated" href="mailto:MPRIS@lists.freedesktop.org">MPRIS@lists.freedesktop.org</a>
<a class="moz-txt-link-freetext" href="http://lists.freedesktop.org/mailman/listinfo/mpris">http://lists.freedesktop.org/mailman/listinfo/mpris</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>