<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <br>
    <div class="moz-cite-prefix">On 06/07/2014 01:16 AM, Aline Manera
      wrote:<br>
    </div>
    <blockquote cite="mid:5391F775.8050909@linux.vnet.ibm.com"
      type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      <div class="moz-cite-prefix">On 06/06/2014 03:49 AM, Hongliang
        Wang wrote:<br>
      </div>
      <blockquote
        cite="mid:1402037382-6455-1-git-send-email-hlwang@linux.vnet.ibm.com"
        type="cite">
        <pre wrap="">Make guest cdrom edit UI consistent with guest interface edit UI. The key point
is to edit cdrom properties in place. Because there is only one property can be
updated for a cdrom, it works fine this way. Though another inconsistence comes
up that in the same storage tab, there is also lines for disks, which have more
editable properties and it's not that easy to make all of these properties stay
in one line. So for disks, possibly we still need provide another window to let
users update properties.
  So here are 2 choices:
  C1) Apply this PATCH v2 to keep cdrom consistent with interface
  C2) Apply PATCH v1 to keep cdrom consistent with disk

  Both are OK for me.

v1 -&gt; v2:
  2a) Made updating cdrom properties in place
     (Aline's comment)

Hongliang Wang (2):
  Adjust Guest Edit Storage Tab Styles
  Remove Unused Files

 ui/css/theme-default/guest-cdrom-edit.css     |  57 ------------
 ui/css/theme-default/guest-edit.css           |  99 +++++++-------------
 ui/images/theme-default/guest-icon-sprite.png | Bin 6748 -&gt; 0 bytes
 ui/js/src/kimchi.guest_cdrom_edit_main.js     |  85 ------------------
 ui/js/src/kimchi.guest_edit_main.js           | 125 +++++++++++++++++++++-----
 ui/pages/guest-cdrom-edit.html.tmpl           |  70 ---------------
 ui/pages/guest-edit.html.tmpl                 |  57 +++++++-----
 7 files changed, 166 insertions(+), 327 deletions(-)
 delete mode 100644 ui/css/theme-default/guest-cdrom-edit.css
 delete mode 100644 ui/images/theme-default/guest-icon-sprite.png
 delete mode 100644 ui/js/src/kimchi.guest_cdrom_edit_main.js
 delete mode 100644 ui/pages/guest-cdrom-edit.html.tmpl

</pre>
      </blockquote>
      <br>
      <font face="DejaVu Sans Mono">We also need to keep consistence in
        the "+"</font> button.<br>
      In the network tab, it adds a new line to the list and user change
      the values accordingly.<br>
      And in the storage tab, a new dialog is displayed to collect user
      input.<br>
      <br>
      Could you send a separate patch for that?<br>
    </blockquote>
    Hi Aline, as I was adding this feature, I found it's different from
    interface that for storage, there is not only CDROMs, but also
    disks. There are more settings for disks so I suggest retain the
    pop-up window for attaching devices.<br>
    <br>
  </body>
</html>