search for: __title

...b/nodeadmin/configscreen.py index 02ab5b4..7a23325 100644 --- a/nodeadmin/configscreen.py +++ b/nodeadmin/configscreen.py @@ -37,6 +37,9 @@ class ConfigScreen: self.__libvirt = LibvirtWorker() self.__vm_config = VirtManagerConfig() + def get_title(self): + return self.__title + def get_hal(self): return self.__hal @@ -83,7 +86,7 @@ class ConfigScreen: screen = SnackScreen() elements = self.get_elements_for_page(screen, self.__current_page) # TODO: need to set the form height to the number of elements on the page -...

...creen.py index 0282eee..f214aea 100644 --- a/nodeadmin/configscreen.py +++ b/nodeadmin/configscreen.py @@ -77,8 +77,9 @@ class ConfigScreen: active = True while active and (self.__finished == False): screen = SnackScreen() - gridform = GridForm(screen, self.__title, 1, 4) elements = self.get_elements_for_page(screen, self.__current_page) + # TODO: need to set the form height to the number of elements on the page + gridform = GridForm(screen, self.__title, 1, 10) current_element = 0 for element in el...

...just looking for some feedback on the direction I'm going. The code won't get all the way to the define stage since I'm in the middle of retrofitting it to use virtinst instead of a home spun node definition.

This version of the patch includes feedback from jboggs at redhat.com, including fixes to the BuildRequires and Requires in the spec file.

This patch obsoletes any previous ones. This is an upstream candidate, so I'm looking for feedback so we can push this and start using it.

This version fixes a few bugs found by jboggs. It also includes some logging facilities that need to be fleshed out more.

This patch is ready for pushing upstream.

The previous patch was missing this file. Resending.