[OpenLayers-Trac] [OpenLayers] #24: Incorporate OpenLayers.Control.ScaleBar

OpenLayers trac at openlayers.org
Mon Jan 7 19:52:31 EST 2008


#24: Incorporate OpenLayers.Control.ScaleBar
----------------------------+-----------------------------------------------
  Reporter:  sderle         |       Owner:  tschaub    
      Type:  feature        |      Status:  assigned   
  Priority:  major          |   Milestone:  2.6 Release
 Component:  Control.Scale  |     Version:             
Resolution:                 |    Keywords:             
     State:  Review         |  
----------------------------+-----------------------------------------------
Comment (by tschaub):

 Replying to [comment:20 ahocevar]:
 >  * The control should have a div property (like e.g.
 Control.!ControlPanel) to allow for placing the scalebar in an existing
 container (outside the map).

 Thanks to inheritance, we get this for free.  I've updated the scalebar-
 thin.html example to show use with a custom "div" and added docs to make
 it clearer that this is possible (see r5567).

 >  * tsep and dsep in the formatNumber function should be configurable
 like measurementProperties to allow for locale-dependant output.

 I think this makes sense.  However, (as we agree) the formatNumber stuff
 belongs elsewhere, so I'm not making these API properties of this control.

 >  * the DOM element title (line 286 in !ScaleBar.js) should not be hard
 coded to start with "scale 1:". This should also be made configurable.

 Also a good suggestion.  I've added a scaleText property to the control.

 Updated patch and sandbox reflects these changes.

-- 
Ticket URL: <http://trac.openlayers.org/ticket/24#comment:23>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer


More information about the Trac mailing list