It seems to give more modern look to the dialog, but it requires Gtk+
3.12, thus I will am keeping this commit separated.
On the glade UI file, we get rid of the GtkAlignment object that was
used to put some space between the tree view and dialog buttons. The
"Select ISO" label is gone too.
Signed-off-by: Eduardo Lima (Etrunko)
---
po/POTFILES.in | 2 +
src/Makefile.am| 3 +
src/remote-viewer-iso-list-dialog.c| 106
src/remote-viewer-iso-list-dialog.h| 58
When user presses the Refresh button in ISO dialog, the list is cleared, and
currently, the only way it is informed of the new list is by the notify signal.
The same applies when an error occurs while trying to change the current ISO.
Signed-off-by: Eduardo Lima (Etrunko)
---
I have pushed the first two patches of the series because they were
already acknowledged and were pretty much self-contained.
I tried to replace GtkTreeView in favor of new GtkListBox, but after some
painful work, I decided to drop it because it is not possible to
reproduce the exact same
Signed-off-by: Eduardo Lima (Etrunko)
---
src/remote-viewer-iso-list-dialog.c| 180 -
src/resources/ui/remote-viewer-iso-list.ui | 5 +-
2 files changed, 181 insertions(+), 4 deletions(-)
diff --git a/src/remote-viewer-iso-list-dialog.c
Signed-off-by: Eduardo Lima (Etrunko)
---
src/resources/ui/virt-viewer.ui | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/resources/ui/virt-viewer.ui b/src/resources/ui/virt-viewer.ui
index 6e3c5ad..af3ae46 100644
---
With this commit, we finish cleaning up ovirt foreign menu code, which
only deals with data, leaving the UI bits to be handled properly in the
new ISO list dialog.
This patch also updates remote-viewer to reflect the change.
Signed-off-by: Eduardo Lima (Etrunko)
---
The OvirtForeignMenu pointer is needed by the new ISO list dialog, and
we make it acessible via property to avoid interdependency between
objects.
Signed-off-by: Eduardo Lima (Etrunko)
---
src/remote-viewer-iso-list-dialog.c | 31 +++
With the new ISO dialog, the user triggers the refresh manually.
Signed-off-by: Eduardo Lima (Etrunko)
---
src/ovirt-foreign-menu.c | 23 +++
1 file changed, 3 insertions(+), 20 deletions(-)
diff --git a/src/ovirt-foreign-menu.c
Signed-off-by: Eduardo Lima (Etrunko)
---
src/ovirt-foreign-menu.c | 49 ++--
src/ovirt-foreign-menu.h | 5 -
2 files changed, 39 insertions(+), 15 deletions(-)
diff --git a/src/ovirt-foreign-menu.c b/src/ovirt-foreign-menu.c
Signed-off-by: Eduardo Lima (Etrunko)
---
src/resources/ui/virt-viewer.ui | 1 +
src/virt-viewer-window.c| 24
2 files changed, 25 insertions(+)
diff --git a/src/resources/ui/virt-viewer.ui b/src/resources/ui/virt-viewer.ui
index
On 07/28/2016 05:20 AM, Daniel P. Berrange wrote:
> This latest virt-manager failure looks genuine:
>
> On Thu, Jul 28, 2016 at 09:12:18AM +, c...@centos.org wrote:
>> --- tests/cli-test-xml/compare/virt-install-kvm-f14-url.xml
>> +++ Generated Output
>> @@ -50,7 +50,7 @@
>>
>>
From: Chen Hanxiao
Signed-off-by: Chen Hanxiao
---
ui/snapshots.ui | 37 +++--
virtManager/snapshots.py | 3 ++-
2 files changed, 21 insertions(+), 19 deletions(-)
diff --git a/ui/snapshots.ui
13 matches
Mail list logo