[GRASS-dev] [bug #5042] (grass) v.digit: Settings pop-up off the screen

[Glynn Clements]

Hamish wrote:

> > this bug's URL: http://intevation.de/rt/webrt?serial_num=5042
> > ---------------------------------------------------------------------
> > 
> > http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=305247
> > 
> > > On a 800x600 monitor, the Settings pop-up window comes up always
> > > partially off the screen and has to be manually moved back.
> > >
> > > Besides, the rightmost main menu Exit button is also half cut off.
> > 
> > Does this refer to placement or the the windows size?
> > GRASS is not doing placement.
> > 
> > I assume that you mean size: in fact the window exceeds
> > 800 pixels in widths which should be somehow fixed (smaller
> > icons?).
> > cc GRASS bugtracker.
> 
> xwininfo show the window has:
>   Width: 813
>   Height: 146
> 
> So we don't need much.
> 
> There are 22 icons across, most are 32x32. Some are smaller, e.g.
> zoom.*.gif. Most could be cropped or edited to 30x30 without much
> damage.
> 
> Alternate, maybe better: move first 11 icons below last 11 (natural
> break there), this results in a 2x11 block of icons.
> 
> It's set at line 76 of vector/v.digit/toolbox.tcl, but I don't know
> enough tcl to make it into two rows. help?

The ButtonBox widget only supports a single row; you need two
separate ButtonBox widgets, i.e. change:

	set bbox [ButtonBox .bbox -spacing 1 -padx 1 -pady 1] 
	pack $bbox -side top -anchor w 
to:
	set bbox1 [ButtonBox .bbox1 -spacing 1 -padx 1 -pady 1] 
	set bbox2 [ButtonBox .bbox2 -spacing 1 -padx 1 -pady 1] 
	pack $bbox1 $bbox2 -side top -anchor w 

and change "$bbox add ..." to "$bbox1 add ..." or "$bbox2 add ...".

-- 
Glynn Clements <glynn at gclements.plus.com>