[node-patches] Change in ovirt-node[master]: disable continue button if Free space was negative.

[rbarry at redhat.com]

Ryan Barry has posted comments on this change.

Change subject: disable continue button if Free space was negative.
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/27955/1/src/ovirt/node/installer/core/storage_vol_page.py
File src/ovirt/node/installer/core/storage_vol_page.py:

Line 109:                                                    _("Continue"),
Line 110:                                                    enabled=False)])
Line 111:             else:
Line 112:                 page.buttons.extend([ui.SaveButton("button.next",
Line 113:                                                    _("Continue"))])
Since ui_content is only called on a page refresh, it's probably better to disable the button in on_change, somewhere in the block after "if changes.contains_any(size_keys)..."

Throwing InvalidData will also disable the continue button.
Line 114: 
Line 115:         return page
Line 116: 
Line 117:     def on_change(self, changes):


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7f8dcfa7b980b5b272fc5845b3f228af7fe4f21e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: hadong <hadong0720 at gmail.com>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes