[GRASS-dev] Re: [GRASS GIS] #1457: Zoom in Georectify Does not Work

[GRASS GIS]

#1457: Zoom in Georectify Does not Work
-----------------------------+----------------------------------------------
 Reporter:  jjrambo          |       Owner:  grass-dev@…              
     Type:  defect           |      Status:  new                      
 Priority:  normal           |   Milestone:  7.0.0                    
Component:  wxGUI            |     Version:  svn-trunk                
 Keywords:  Zoom Georectify  |    Platform:  Linux                    
      Cpu:  x86-32           |  
-----------------------------+----------------------------------------------

Comment(by martinl):

 Replying to [comment:5 mmetz]:
 > > yes, see r47987. I think there is no need to disable this feature in
 georectifier(?) For quick `gcpmapdisp.py` code should be updated or
 rewrite code as I suggested and use common super-class for also for
 `mapdisp.MapFrame`. This avoid such bugs in the future and also made life
 easier for implementing similar tools, like wxGUI i.class, etc.
 >
 > I common superclass could be difficult, mainly because of the differing
 window layout managed by AUI. The status bar of the georectifier has
 entries different from the regular map display, i.e. some entries present
 in the status bar of regular map display are not present in the status bar
 of the georectifier and vice versa, e.g. "Go to GCP No." The georectifier
 uses the buffered window of wxGUI as is, but this is tightly interwoven
 with mapdisp, i.e. you do not get the one without the other one. That
 makes things difficult for georectifier, some wx.class etc. which also
 draw maps on canvas but interact somewhat differently with the maps.

 OK, I didn't study the code so deeply, anyway I see some overlap in
 mapdisp.py and gcpmapdisplay.py. Speaking about statusbar items, there are
 some which are common, some of them differs. Common items could be handled
 by this super-class.

 Martin

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/1457#comment:6>
GRASS GIS <http://grass.osgeo.org>