[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] colored menu
From: |
Yoshinori K. Okuji |
Subject: |
Re: [PATCH] colored menu |
Date: |
Mon, 24 Dec 2007 20:46:29 +0100 |
User-agent: |
KMail/1.9.4 |
On Sunday 23 December 2007 22:09, Robert Millan wrote:
> This patch adds a nice looking colored menu like the one you'd obtain on
> GRUB Legacy with "color cyan/blue white/blue" command. Screenshot is
> attached as well.
>
> A pair of notes:
>
> - The *_setcolor function stubs are being removed from some terminals
> because grub_setcolor() from kernel already skips undefined *_setcolor
> implementations so there's no need for an empty stub. In fact, not
> needed for this patch but adds confusion (I had to inspect all them to see
> if *_getcolor() had to be added).
>
> - I know that hardcoding colors is not so nice, but I was unsure how
> would the selection interface have to look like (and lacking time to
> implement it), and in comparison with defaulting to hardcoded 0x07 (grey on
> black) I see it as an improvement.
I still prefer to keep the default as it is. I like that a user (or a
distributor) would modify the colors in grub.cfg.
Anyway, this will be a temporary hack, right? I'd love to see a fancier
interface built in. Mmh, I really need to allocate time to implement one...
Okuji