On Tue, 11 Mar 2008 16:10:42 +0100
Dominik Vogt <[EMAIL PROTECTED]> wrote:
> > MAX_VERTICAL_SPACING and MIN_VERTICAL_SPACING are not defined in
> > the patch. Anyway, MIN_VERTICAL_SPACING should be just 0 and
> > MAX_VERTICAL_SPACING the same value that is used for
> > MAX_MENU_BORDER_WIDTH.
>
> Ah, these values are already defined. It's better to use a new
> constant MAX_MENU_MARGIN (in libs/defaults.h).
>
Yes. I set it to 50, same value that MAX_MENU_BORDER_WIDTH as you
suggested.
> @Victor: I wanted to say that it might make sense to have a
> style similar to ItemFormat for the vertical layout too.
I would probably be able to implement this that way if you really think
it's better. The obvious benefits are:
1.- consistency
2.- leave the doors open to implement options to align the text
vertically (I don't know how much could it take me to
implement that, maybe it's trivial, I'll have to look).
If you prefer it that way, let me know and I'll try to do my best.
Thanks so much for all the input, it's much appreciated.
--
Jesús Guerrero <[EMAIL PROTECTED]>
