Ticket #42 (closed defect: fixed)

Opened 7 months ago

Last modified 7 months ago

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

Reported by: marisn Assigned to: grass-dev@lists.osgeo.org
Priority: blocker Milestone: 6.3.0
Component: default Version: svn-trunk
Keywords: nviz Cc:
Platform: CPU:

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

nviz_fly_panel.path (4.1 kB) - added by marisn on 02/09/08 10:54:04.
Correct patch. Also fixes GForge issue #540 and other small annoyances

Change History

02/09/08 10:54:04 changed by marisn

  • attachment nviz_fly_panel.path added.

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

02/09/08 13:01:19 changed by cmbarton

  • status changed from new to closed.
  • resolution set to fixed.

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

02/09/08 13:04:56 changed by 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?

02/09/08 13:28:53 changed by cmbarton

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

02/09/08 18:44:46 changed by 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