[PATCH 2/2] Cleaning up some of the TUI storage code

Chris Lumens clumens at redhat.com
Tue May 7 21:08:17 UTC 2013


> @@ -313,14 +324,14 @@ class AutoPartSpoke(NormalTUISpoke):
>              # Default to clearing everything.
>              self.clearPartType = CLEARPART_TYPE_ALL
>  
> -        self.parttypelist = sorted(parttypes.keys())
>          for parttype in self.parttypelist:
>              c = CheckboxWidget(title="%i) %s" % (self.parttypelist.index(parttype) + 1,
>                                                   parttype),
> -                               completed=(parttypes[parttype] == self.clearPartType))
> +                               completed=(PARTTYPES[parttype] == self.clearPartType))
>              self._window += [c, ""]
>  
> -        message = _("Installation requires partitioning of your hard drive.  Select what space to use for the install target.")
> +        message = _("Installation requires partitioning of your hard drive.  \
> +                     Select what space to use for the install target.")
>  
>          self._window += [TextWidget(message), ""]
>  

While you're modifying stuff here, could you change "install" to
"installation" in message?  I hate how that reads, and have changed it
all over the place in the GUI.

- Chris


More information about the anaconda-patches mailing list