[node-patches] Change in ovirt-node[master]: Tweak the upgrade page

rbarry at redhat.com rbarry at redhat.com
Thu Dec 19 16:03:55 UTC 2013


Ryan Barry has uploaded a new change for review.

Change subject: Tweak the upgrade page
......................................................................

Tweak the upgrade page

We showed an error/notification message as a Label field, and
a divider pushed the buttoms of the bottom of the screen if there
was a long message from a ConfirmedEntry's security check.

Remove the divider so the buttons can be seen, and make the label
a notice instead.

Change-Id: Iab1d940447ef456d583cdc25d043ee81967ae747
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1039255
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/installer/core/upgrade_page.py
1 file changed, 2 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/71/22571/1

diff --git a/src/ovirt/node/installer/core/upgrade_page.py b/src/ovirt/node/installer/core/upgrade_page.py
index 5d28f5f..bef98b9 100644
--- a/src/ovirt/node/installer/core/upgrade_page.py
+++ b/src/ovirt/node/installer/core/upgrade_page.py
@@ -59,9 +59,8 @@
                                _("Current Password:")),
               ui.Divider("divider[1]"),
               ui.ConfirmedEntry("upgrade.password", _("Password:"),
-                                is_password=True),
-              ui.Divider("divider[2]"),
-              ui.Label("current_password.info", ""),
+                                is_password=True, min_length=3),
+              ui.Notice("current_password.info", ""),
               ui.Label("password.info", self.__no_new_password_msg)
               ]
         page = ui.Page("password", ws)


-- 
To view, visit http://gerrit.ovirt.org/22571
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iab1d940447ef456d583cdc25d043ee81967ae747
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>



More information about the node-patches mailing list