[PATCH] Verify user ready to proceed with installation in TUI. (#975793)

Brian C. Lane bcl at redhat.com
Tue Jul 23 16:58:37 UTC 2013


On Tue, Jul 23, 2013 at 08:53:12AM -0400, Samantha N. Bueno wrote:
> On Mon, Jul 22, 2013 at 05:17:59PM -0700, Brian C. Lane wrote:
> > On Mon, Jul 22, 2013 at 08:32:11AM -0400, Samantha N. Bueno wrote:
> > > Another for rhel7-branch as well.
> > > 
> > > I feel like a better-worded message could be shown to users, so please
> > > feel free to offer suggestions.
> > 
> > Wouldn't it be clearer to change the prompt? Instead of 'c' to continue
> > have it say "'s' to start install"
> 
> Yeah, you have a point. I thought the prompt was at risk of looking a
> little too crowded, but here's a screenshot with the option added in:
> http://sbueno.fedorapeople.org/images/975793.jpg
> 
> What do you think?

I was thinking it would replace 'c' continue. If both of them are there
what does 'c' do?

> 
> Even with a clear "start installation" option though, I feel having the
> yes/no question present is still a good idea, just because it's easy
> enough to accidentally hit the wrong key.

I agree with Chris that we shouldn't add the confirmation. If starting
the install requires s instead of c at this point it makes it harder to
do it on accident.

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 482 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/anaconda-patches/attachments/20130723/48c3d3da/attachment.sig>


More information about the anaconda-patches mailing list