All of lore.kernel.org
 help / color / mirror / Atom feed
From: Carles Pina i Estany <carles@pina.cat>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: menu loop (patch)
Date: Sun, 20 Jul 2008 00:06:40 +0200	[thread overview]
Message-ID: <20080719220640.GA4419@pina.cat> (raw)
In-Reply-To: <20080719103446.047a33e6@gibibit.com>


Hello,

On Jul/19/2008, Colin D Bennett wrote:
> On Sat, 19 Jul 2008 17:40:43 +0200
> Robert Millan <rmh@aybabtu.com> wrote:
> 
> > On Thu, Jul 17, 2008 at 11:22:27PM +0200, Carles Pina i Estany wrote:

> > > I'm a Grub user (thanks for your nice work!) and I always wanted to
> > > have a menu that "loops". Like, if you press down and you are in
> > > the last option it goes to the first one, and if you press up but
> > > you are int he first option goes to the last.

[...]

> > What does everyone else think?
> 
> I think that wrapping around the menu with the arrow keys *can* be
> useful in some situations, but in general it seems like we should try
> to be consistent with the common UI behavior that is expected, at
> least by default.  We could support a setting such as "set menuwrap=1"
> which would enable this feature for users who care about it.  Then the
> users who don't care won't miss it, but the users who do care can
> still have it by adding a single line to grub.cfg.

in one way, add an options for this looks a bit like "overkill" (as
Robert said). In the other hand, I have my hopes that distributions
would like it and (mainly) all of them would add this option by default
:-) so could be a nice way to know what some people think.

> > Colin, does this affect your graphical menu work in some way?  I
> > suppose the same situation applies the same way to the upcoming new
> > menu.  Or perhaps this can be made more flexible and be toggled by
> > some CSS magic?
> 
> This would be no problem at all; it's essentially identical to how the
> text mode menu from normal.mod works.  It could go either way, and
> certainly would be easy to make it a setting in the theme file (or more
> likely in a user-configurable place such as grub.cfg (?) -- since we
> might not expect most users to modify theme files but only choose which
> one to use).
> 
> Actually my graphical menu currently *does* wrap around, I guess it

I'm new in this list and in Grub2 (I hvae not used, actually). Are you
saying that there is some graphical menu that does wrap?
I've read some opinions here against wrapping, but if the graphical ones
do it, in my (non-experted) opinion would be nice to have the wrapping
in text and graphic.

> seemed logical to me at the time I wrote the code!  8-)  As you can
> tell, I am not firmly set on either wrapping or not wrapping.

my workmate wanted it, i wanted and I've spoke with my brother and he
really wanted. But maybe it's because brother-love :-) (this is not
proving anything, of course)

-- 
Carles Pina i Estany		GPG id: 0x8CBDAE64
	http://pinux.info	Manresa - Barcelona



  parent reply	other threads:[~2008-07-19 22:06 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-17 21:22 menu loop (patch) Carles Pina i Estany
2008-07-19 15:40 ` Robert Millan
2008-07-19 17:21   ` Carles Pina i Estany
2008-07-19 19:52     ` Robert Millan
2008-07-19 22:12       ` Carles Pina i Estany
2008-07-19 23:26     ` Isaac Dupree
2008-07-19 23:31       ` Carles Pina i Estany
2008-07-19 23:45         ` Isaac Dupree
2008-07-20  4:24         ` Colin D Bennett
2008-07-19 17:34   ` Colin D Bennett
2008-07-19 19:58     ` Robert Millan
2008-07-19 20:31       ` Yoshinori K. Okuji
2008-07-19 22:06     ` Carles Pina i Estany [this message]
2008-07-19 22:15       ` Robert Millan
2008-07-19 22:28       ` Colin D Bennett
  -- strict thread matches above, loose matches on Subject: below --
2008-07-20  6:21 chaac
2008-07-20  8:49 ` Carles Pina i Estany
2008-07-22 21:40 ` Robert Millan
2008-08-05  7:15   ` Vesa Jääskeläinen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080719220640.GA4419@pina.cat \
    --to=carles@pina.cat \
    --cc=grub-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.