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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).