Re: Key layout patch

[Daniel J S. <dan...@ie...>]

Petr Mikulik wrote:
>> I think the key-cleanup
> 
> 
> Yes, that's interesting. Please update it for current cvs.
> There, I propose to change
>     ... outside | below | <position>}
> to
>     outside | below | at <position>}
> 
> as giving raw numbers is always confusing.

OK.  It's changed, consistent with "labels" placement:

  Syntax:
        set key {on|off} {default}
                {{inside | outside} | {at <position>} | {above | below}}
                {left | right | center} {top | bottom | center}
[...]

The output of 'key.dem' is at:

http://www.acer-access.com/~ds...@ac.../gnuplot/key.pdf

I'll be leaving town shortly, so I've put an up-to-date patch on S.F.  However, 
there is one detail left.  It has to do with a discussion between Hans and I 
about manually set margins.  I couldn't recall the exact issue, but I have all 
the old emails.  (The problem is that ybot is a global variable set all over the 
place.  This fix was right in the middle of some code that gets tossed by the 
new patch.)

Dan


#if 0
     /* Dan Sebald 21jun2005:  Hans had added this hunk of code after
      * a new key patch was created.  It was placed somewhere that made
      * no sense in the new patch.  Sorry if this has created a bug,
      * but I can't exactly recall what the issue was.  Perhaps if
      * that bug crops up again we can find a proper home for this code.
      */
		/* HBB 20040725: leave manually set bmargin alone */
		if (bmargin < 0) {
		    ybot += key_entry_height * key_rows
			+ (int) (t->v_char * (ktitl_lines + 1));
		    ybot += (int) (key->height_fix * t->v_char);
		}
#endif