#42 closed defect (fixed)
NVIZ panels fail when switching view mode from fly back to eye or center
Reported by: | marisn | Owned by: | |
---|---|---|---|
Priority: | blocker | Milestone: | 6.3.0 |
Component: | Default | Version: | svn-trunk |
Keywords: | nviz | Cc: | |
CPU: | Unspecified | Platform: | Unspecified |
Description
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)
Attachments (1)
Change History (5)
by , 17 years ago
Attachment: | nviz_fly_panel.path added |
---|
comment:1 by , 17 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
I don't know whether Maris' patch was applied, but this bug is now fixed in SVN
comment:2 by , 17 years ago
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?
comment:3 by , 17 years ago
I applied other fixed in Maris' patch with a small modification that avoided re-introducing the view-switching bug.
comment:4 by , 17 years ago
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
Correct patch. Also fixes GForge issue #540 and other small annoyances