Messages in this thread | | | Date | Tue, 25 Nov 2008 09:11:53 +0100 | From | Sam Ravnborg <> | Subject | Re: [PATCH] ncurses based config V2 |
| |
Hi Nir.
> >> [ ] (U)TS namespace > >> [ ] (I)PC namespace > >> [ ] (U)ser namespace (EXPERIMENTAL) > >> [ ] (P)ID Namespaces (EXPERIMENTAL) > > > > This looks like some ancient stuff and it must be a better way to > > show the short-cuts. > > This was the best I could come up with, as ncurses does not let you > use color attributed for menu items. I'll try a different scheme, > where the first capitalized letter is the shortcut. What do u say?
Better than the above - let me see it in action and I will judge. I had preferred something with a colored highlight - but if ncurses do not support that then I think first catital letter i ok.
Alternative is to use first letter in the prompt and accept the ambiguity.
> > I would really appreciate if you could look into the above issues. > > If you do a repost with this fixed I will try to do a proper review, > > including code-review next time. > > Sure. I'll have a patch ready in a few days.
Great - loks forward to see it.
Sam
| |