grub-devel.gnu.org archive mirror
 help / color / mirror / Atom feed
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



  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).