[master/rhel7-branch] Make it obvious user is going to begin installation in TUI. (#975793)

Samantha N. Bueno sbueno+anaconda at redhat.com
Fri Sep 20 19:46:19 UTC 2013


On Fri, Sep 20, 2013 at 11:28:37AM -0400, Vratislav Podzimek wrote:
> On Fri, 2013-09-20 at 10:45 -0400, Samantha N. Bueno wrote:
> > Re-post of a patch from forever ago that needed a little work. This
> > should only require very minimal tweaking for rhel7-branch.
> > 
> > This tweaks the global default prompt for the new option ('b' to
> > begin installation), so it should now be very clear to users that they
> > are about to kick off installation.
> > This also adds a default prompt for spokes, as many of them need to have
> > the 'c' to continue option, and it is probably best to just have 'b' to
> > begin installation on the summary hub anyhow.
> > 
> > End result looks like this:
> > http://sbueno.fedorapeople.org/images/975793-revised.jpg
> > 
> > Resolves: rhbz#975793
> > ---
> >  pyanaconda/ui/tui/hubs/__init__.py   |  2 +-
> >  pyanaconda/ui/tui/simpleline/base.py |  6 +++---
> >  pyanaconda/ui/tui/spokes/__init__.py | 10 ++++++++--
> >  pyanaconda/ui/tui/spokes/network.py  |  3 +++
> >  4 files changed, 15 insertions(+), 6 deletions(-)
> > 
> > diff --git a/pyanaconda/ui/tui/hubs/__init__.py b/pyanaconda/ui/tui/hubs/__init__.py
> > index 8f219c3..be78b32 100644
> > --- a/pyanaconda/ui/tui/hubs/__init__.py
> > +++ b/pyanaconda/ui/tui/hubs/__init__.py
> > @@ -109,7 +109,7 @@ class TUIHub(TUIObject, common.Hub):
> >              # If we get a continue, check for unfinished spokes.  If unfinished
> >              # don't continue
> >              # TRANSLATORS: 'c' to continue
> > -            if key == _('c'):
> > +            if key == _('b'):
> Don't forget to update the TRANSLATORS comment right above.

Ah right, thanks for that.
 
> Otherwise this looks good to me. Could you just please remind me why we
> don't leave spokes and hubs in general use 'c' and 'continue' and just
> override the SummaryHub's prompt and input methods to use 'b' and
> 'begins installation'? Prompt could be overriden quite easily and the
> input method could replace 'b' with 'c' (or more precisely their
> translations). Wouldn't that work?

There's no better reason logically, I just thought it was nicer to have
things conform to defaults, even if it did involve adding a new default
(e.g. default spoke prompt). I can re-write it to just have the custom
prompt and input functions in the SummaryHub if you think that's better;
I could go either way.

Samantha


More information about the anaconda-patches mailing list