NVIZ panels fail when switching view mode from fly back to eye or center

[marisn]

Switching view mode from fly to eye or center fails and leaves nviz in unusable state.

window ".middle.panelarea.panels.main.midf.height" isn't packed
window ".middle.panelarea.panels.main.midf.height" isn't packed
    while executing
"pack $XY -side left -before $Nv_(HEIGHT_SLIDER)"
    (procedure "change_display" line 46)
    invoked from within
"change_display 1"
    invoked from within
".middle.panelarea.panels.main.midt.b1 invoke"
    ("uplevel" body line 1)
    invoked from within
"uplevel #0 [list $w $cmd]"
    (procedure "tk::CheckRadioInvoke" line 3)
    invoked from within
"tk::CheckRadioInvoke .middle.panelarea.panels.main.midt.b1"
    (command bound to event)

[marisn]

Correct patch. Also fixes GForge issue #540 and other small annoyances

[cmbarton]

I don't know whether Maris' patch was applied, but this bug is now fixed in SVN

[marisn]

Should I open new bugs for other issues, that where fixed by my patch and are left open by  http://trac.osgeo.org/grass/changeset/30034?

[cmbarton]

I applied other fixed in Maris' patch with a small modification that avoided re-introducing the view-switching bug.

[hamish]

In a quick test, the latest SVN works for me & Spearfish's elevation.dem.

Maris:

Should I open new bugs for other issues, that where fixed by my patch and are left open by  http://trac.osgeo.org/grass/changeset/30034?

Yes, one patch/ticket per issue, otherwise it is very likely they will be forgotten.

hint: just type a "r" in front of the revision number and trac will automatically link it. (r30034)

Hamish