[Mapguide-trac] [mapguide-trac] #1834: Measure tool should provide a unit selection option

MapGuide Open Source trac_mapguide at osgeo.org
Mon Jun 18 08:35:59 PDT 2012


#1834: Measure tool should provide a unit selection option
---------------------------+------------------------------------------------
   Reporter:  trevorwekel  |       Owner:  trevorwekel      
       Type:  enhancement  |      Status:  new              
   Priority:  low          |   Milestone:  2.4              
  Component:  AJAX Viewer  |     Version:  2.2.0            
   Severity:  trivial      |    Keywords:  measure, haspatch
External_id:               |  
---------------------------+------------------------------------------------

Comment(by jng):

 There is one small bug I've found in measureui.templ

 The SetUnits() function has the unit assignment in the wrong order. The
 line
 {{{
 units = GetParent().GetMapFrame().IsEnglishUnits()? "km": "mi";
 }}}
 should be the other way round.
 {{{
 units = GetParent().GetMapFrame().IsEnglishUnits()? "mi": "km";
 }}}

 This ensures the default unit selection in the Measure UI lines up with
 whatever choice was set in the viewer options.

 Other than that, the patch looks fine

-- 
Ticket URL: <http://trac.osgeo.org/mapguide/ticket/1834#comment:6>
MapGuide Open Source <http://mapguide.osgeo.org/>
MapGuide Open Source Internals


More information about the mapguide-trac mailing list