[node-patches] Change in ovirt-node[master]: app: Add to footer the F2 option

dougsland at redhat.com dougsland at redhat.com
Thu Sep 25 14:50:01 UTC 2014


Douglas Schilling Landgraf has posted comments on this change.

Change subject: app: Add to footer the F2 option
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/33372/1/src/ovirt/node/app.py
File src/ovirt/node/app.py:

Line 285:         self.ui.register_hotkey("f12", self.__reload_page)
Line 286:         self.ui.register_hotkey("window resize", self.__check_terminal_size)
Line 287: 
Line 288:         self.ui.header = "\n %s\n" % str(self.product)
Line 289:         self.ui.footer = "Press Esc to quit || F2 to shell rescue"
> This is cosmetic, but I'd probably make it something more like:
Sure, I will just avoid the duplication of Press but I am open to change it.
Line 290: 
Line 291:         # Catch ctrl+c
Line 292:         if not self.args.debug:
Line 293:             def _handler(signum, frame):


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I425414f6ed45c6b253623360bc3d3f8ec9dbd0ce
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes



More information about the node-patches mailing list