<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 8/12/2014 2:41 AM, Cr&iacute;stian Viana
      wrote:<br>
    </div>
    <blockquote cite="mid:53E90E68.9020604@linux.vnet.ibm.com"
      type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      On 11-08-2014 11:22, <a moz-do-not-send="true"
        class="moz-txt-link-abbreviated"
        href="mailto:shaohef@linux.vnet.ibm.com">shaohef@linux.vnet.ibm.com</a>
      wrote:<br>
      <blockquote
        cite="mid:1407766943-9178-1-git-send-email-shaohef@linux.vnet.ibm.com"
        type="cite">
        <pre wrap="">From: ShaoHe Feng <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:shaohef@linux.vnet.ibm.com">&lt;shaohef@linux.vnet.ibm.com&gt;</a>

We don not need a special "save" button for permission form.

All form in guest edit tab can share the same "save" button.

also, we will add a password form, it will also share this button.

Signed-off-by: ShaoHe Feng <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:shaohef@linux.vnet.ibm.com">&lt;shaohef@linux.vnet.ibm.com&gt;</a>
Signed-off-by: Simon Jin <a moz-do-not-send="true" class="moz-txt-link-rfc2396E" href="mailto:simonjin@linux.vnet.ibm.com">&lt;simonjin@linux.vnet.ibm.com&gt;</a>

</pre>
      </blockquote>
      Why are we using a save button for each tab in that dialog? That's
      definitely not a common user experience. One would expect that a
      "Save" button in that dialog would apply all changes done in the
      dialog, not on that tab alone. What if the user wants to change
      the VM name and permission (and the password, after the next
      patch)? They will need to open the "Edit Guest" dialog, changing
      the value they want to change, and then saving that value alone. <i>Three

        times.</i> That's not a good flow.<br>
      <br>
      IMO, we should have only one "Save" button which, when clicked,
      applies all the changes done in that dialog. Not a button which
      does different things depending on which tab the user is on.<br>
    </blockquote>
    <br>
    1. Right now, there are already many options when editing VM, and I
    believe there will be more and more.<br>
    &nbsp;&nbsp;&nbsp; Each time user edit VM, they will only edit one or two options(a
    small portion of all), save all the options back is inefficient and
    go against user's intension.<br>
    2. The design need to be changed to save what user has changed and
    allow user to change at a fine granularity, I noticed that
    virt-manager did a good job here.<br>
    &nbsp;&nbsp;&nbsp; In virt-manager, user can update anything in a quite fine
    granularity and the 'apply' button feels quite make sense to
    indicate that it just update what user is changing.<br>
    <br>
    &nbsp;&nbsp;&nbsp; <img src="cid:part5.07090303.04030202@linux.vnet.ibm.com"
      alt=""><br>
    <br>
    <blockquote cite="mid:53E90E68.9020604@linux.vnet.ibm.com"
      type="cite"> <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
Kimchi-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Kimchi-devel@ovirt.org">Kimchi-devel@ovirt.org</a>
<a class="moz-txt-link-freetext" href="http://lists.ovirt.org/mailman/listinfo/kimchi-devel">http://lists.ovirt.org/mailman/listinfo/kimchi-devel</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>