Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#42 closed defect (fixed)

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

Reported by: marisn Owned by: grass-dev@…
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)

nviz_fly_panel.path (4.1 KB ) - added by marisn 16 years ago.
Correct patch. Also fixes GForge issue #540 and other small annoyances

Download all attachments as: .zip

Change History (5)

by marisn, 16 years ago

Attachment: nviz_fly_panel.path added

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

comment:1 by cmbarton, 16 years ago

Resolution: fixed
Status: newclosed

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

comment:2 by marisn, 16 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 cmbarton, 16 years ago

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

comment:4 by hamish, 16 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

Note: See TracTickets for help on using tickets.