Re: Small typo in kernel [current source from git] .config option

[Nir Tzachar]

Hello.

On Wed, Jul 28, 2010 at 9:55 PM, Sam Ravnborg <sam@xxxxxxxxxxxx> wrote:
> On Wed, Jul 28, 2010 at 05:41:47PM +0200, Michal Marek wrote:
>> (adding Nir, who wrote nconfig, to CC).
>>
>> On 28.7.2010 14:17, trapDoor wrote:
>> > On Wed, Jul 28, 2010 at 12:02 PM, Michal Marek <mmarek@xxxxxxx> wrote:
>> >> respectively). Unfortunatelly, there doesn't seem to be a way to
>> >> highlight single letters in an ncurses menu, hence the strange
>> >> cApitalization :(.
>> >
>> > I wonder how is this resolved in other console applications such as
>> > Midnight Commander. Does it have its own implementation for
>> > highlighting characters (it uses coloured mnemonics e.g. in menus) or
>> > it's based on some common Linux/Unix library which perhaps could be
>> > used for the nconfig as well?
>>
>> The problem is that nconfig uses the higher-level ncurses libraries
>> libmenu and libpanel, which make some things easier, but aren't as
>> flexible as using ncurses directly. Patches are welcome of course :-).

The problem does arise from the use of libmenue, which gives you no
control over the rendering of menue items. You can not specify any
attributes for specific menu item's letters.

> The capitilazing is the main reason why I have
> not recommended nconfig to be default.
>
> And I doubt the shortcut letters are used that often.
> One idea could be to drop the shortcut for the individual menus.

I almost never use them and removing them will have the benefit of
cleaning the code.

A different approach may be to reimplement libmenu's logic (recreating
the old lxdialog code), but avoiding this was part of the reason for
writing nconfig....

In any way, I am happy people are starting to use nconfig.
Cheers.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/