From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1KKGL9-00051Q-UB for mharc-grub-devel@gnu.org; Sat, 19 Jul 2008 13:35:27 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KKGL7-0004yY-Nk for grub-devel@gnu.org; Sat, 19 Jul 2008 13:35:25 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KKGL6-0004vo-9L for grub-devel@gnu.org; Sat, 19 Jul 2008 13:35:25 -0400 Received: from [199.232.76.173] (port=52215 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KKGL5-0004vd-VV for grub-devel@gnu.org; Sat, 19 Jul 2008 13:35:24 -0400 Received: from gateway08.websitewelcome.com ([69.56.176.29]:33779) by monty-python.gnu.org with smtp (Exim 4.60) (envelope-from ) id 1KKGL5-0004T9-FS for grub-devel@gnu.org; Sat, 19 Jul 2008 13:35:23 -0400 Received: (qmail 15366 invoked from network); 19 Jul 2008 17:45:33 -0000 Received: from gator297.hostgator.com (74.53.228.114) by gateway08.websitewelcome.com with SMTP; 19 Jul 2008 17:45:33 -0000 Received: from c-67-185-142-228.hsd1.wa.comcast.net ([67.185.142.228]:42841 helo=localhost) by gator297.hostgator.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.68) (envelope-from ) id 1KKGL0-0006em-Hx; Sat, 19 Jul 2008 12:35:18 -0500 Date: Sat, 19 Jul 2008 10:34:46 -0700 From: Colin D Bennett To: The development of GRUB 2 Message-ID: <20080719103446.047a33e6@gibibit.com> In-Reply-To: <20080719154043.GG23778@thorin> References: <20080717212227.GA11725@pina.cat> <20080719154043.GG23778@thorin> X-Mailer: Claws Mail 3.5.0 (GTK+ 2.12.10; i686-pc-linux-gnu) Mime-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/f7b15urXyyd2LaWfGidbo3f"; protocol="application/pgp-signature"; micalg=PGP-SHA1 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - gator297.hostgator.com X-AntiAbuse: Original Domain - gnu.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - gibibit.com X-detected-kernel: by monty-python.gnu.org: Linux 2.6 (newer, 3) Cc: rmh@aybabtu.com Subject: Re: menu loop (patch) X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GRUB 2 List-Id: The development of GRUB 2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 19 Jul 2008 17:35:26 -0000 --Sig_/f7b15urXyyd2LaWfGidbo3f Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Sat, 19 Jul 2008 17:40:43 +0200 Robert Millan wrote: > On Thu, Jul 17, 2008 at 11:22:27PM +0200, Carles Pina i Estany wrote: > >=20 > > Hello, >=20 > Carles! Que n'=C3=A9s de petit el m=C3=B3n ;-) >=20 > > 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. > >=20 > > Attached comes a patch against revision 1718. > >=20 > > Of course, feel free to comment anything about it. I'm open to > > change, fix, etc. > >=20 > > I also don't know if this functionality is not in Grub because you > > thought that was better to not have this functionality.=20 > >=20 > > I've copied/change a few lines (after the patch, lines 425-428 are > > the same than lines 347-350). I don't think that it's needed to > > refactor but if you think so I can do it. >=20 > I'm not sure if this is intentional. Size is not important in this > part of the code, but in other UI menus (e.g. Iceweasel or GNOME) I > notice this is not done. It could also be a nuissance for users who > want to go rapidly to the top or bottom (sure, they can use > PgUp/PgDn, but then aren't we making the behaviour of those keys > inconsistent with arrows?). >=20 > Also, if you (or anyone) can point to a "usability best practices" > reference saying this has been carefuly analized and one of the > options was found to be better, I think this would help us make a > decision. >=20 > 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=3D1" 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. > 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 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. Regards, Colin --Sig_/f7b15urXyyd2LaWfGidbo3f Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.9 (GNU/Linux) iEYEARECAAYFAkiCJbwACgkQokx8fzcGbYfIhACfbmcwycnc3HD8dU5U14/GK/QK qKYAoJeZHnb5+txeJTOyFSIoZ87D34VZ =eMjn -----END PGP SIGNATURE----- --Sig_/f7b15urXyyd2LaWfGidbo3f--