From: Endres Puschner <code@e7p.de>
To: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: Horizontal boot menu
Date: Wed, 01 Jun 2011 18:15:51 +0200 [thread overview]
Message-ID: <4DE665B7.7050007@e7p.de> (raw)
In-Reply-To: <4DE58028.6030406@gmail.com>
On 01.06.2011 01:56, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> On 10.04.2011 15:09, Endres Puschner wrote:
>> Hi, grub-developers,
>>
>> I implemented a complete horizontal extension to the boot menu and
>> some other little features and fixes for extending the gfxmenu boot menu.
>>
> Haven't looked through whole of it but it has 2 serious problems:
Yes, both problems exist, because I coded it mainly for myself.
Even I was not very happy with this part, but I didn't know how to make
it better. Maybe it should be reconsidered after the decision of the
vi-like-keys topic is made, as there was only the predefined user
interaction in the past, for example, up- and down-keys were a very
default method for navigating in every interface.
I think it would be pointless to fix these bugs now while some other
things may change meanwhile.
> 1) Environment variable is used where it's not to be used by scripts. In
> such cases a C variable is more appropriate.
> 2) It doesn't handle the multiterminal case correctly. Even if the menu
> on local console is vertical, the menu on e.g. serial will remain
> vertical and with your code the remote operator will have very confusing
> key settings. It would be nicer if one could have the same key bindings
> for both vertical and horizontal menu. The operation of left/right keys
> can be an alias to up/down. In case it's justifiably impractical you
> would need a new terminal flag which marks a local console. It should be
> set on *_keyboard and console (all platform variants). This could then
> also be used for some things Colin proposed some time ago.
>> I would just describe my added theme parameters for the boot_menu
>> control:
>>
>> - item_width: Optionally specifying a width for each item in
>> horizontal lists, but could also be implemented to work with vertical
>> lists.
>> - item_icon_padding: Padding between icon and box or between icon
>> and other item if there is no item_box.
>> - horizontal: boolean, defining this boot_list as horizontal.
>> - item_pixmap_style: like selected_item_pixmap_style but for non
>> selected entries.
>> - item_pixmap_icon_only: boolean, selected and unselected pixmaps
>> are only shown around the icons and not around the whole item.
>>
>> The scrollbars must be manually turned around to be in horizontal
>> mode, I have uploaded a demo theme as an example:
>>
>> http://data.e7p.de/grub-horizontal-theme.png
>> http://data.e7p.de/grub-horizontal-theme.zip
>>
>> I fixed a bug in the icon-section. The first group in any menuentry
>> was skipped while looking for icons before, so only icons for the
>> second class would be searched.
>>
>> The normal menu is also modified as I wanted to allow to use
>> right/left keys while navigating through the menu while it is
>> horizontal. At menu_init, a new environment variable (orientation) can
>> be set by the used menu module to horizontal for obvious reasons.
>>
>> Best regards,
>> Endres
>>
>>
>> _______________________________________________
>> Grub-devel mailing list
>> Grub-devel@gnu.org
>> http://lists.gnu.org/mailman/listinfo/grub-devel
>
>
>
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel
next prev parent reply other threads:[~2011-06-02 0:28 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-10 19:09 Horizontal boot menu Endres Puschner
2011-05-31 23:56 ` Vladimir 'φ-coder/phcoder' Serbinenko
2011-06-01 16:15 ` Endres Puschner [this message]
2011-06-02 4:09 ` Vladimir 'φ-coder/phcoder' Serbinenko
-- strict thread matches above, loose matches on Subject: below --
2013-05-31 7:02 horizontal " kiran ps
2013-05-31 14:49 ` Lennart Sorensen
2013-05-31 17:42 ` kiran ps
2013-06-01 21:22 ` Gerard Butler
2013-06-07 7:00 ` kiran ps
2013-06-07 7:06 ` kiran ps
2013-06-07 16:26 ` Andrey Borzenkov
2013-06-07 16:30 ` Vladimir 'φ-coder/phcoder' Serbinenko
2013-06-08 7:30 ` Gerard Butler
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=4DE665B7.7050007@e7p.de \
--to=code@e7p.de \
--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.