[OpenLayers-Trac] [OpenLayers] #794: give OpenLayers.Map a resolutions property

OpenLayers trac at openlayers.org
Tue Jul 3 13:07:55 EDT 2007


#794: give OpenLayers.Map a resolutions property
----------------------+-----------------------------------------------------
  Reporter:  tschaub  |       Owner:  tschaub    
      Type:  feature  |      Status:  new        
  Priority:  minor    |   Milestone:  2.5 Release
 Component:  Map      |     Version:  2.4        
Resolution:           |    Keywords:  review     
----------------------+-----------------------------------------------------
Comment (by euzuro):

 patch looks awesome. two things:

  * //OPTIONS - I deliberately set that at two spaces indented so that it
 breaks with the vertical line of all the properties to denote a new
 section. If you hate that, make it four. But otherwise pretty please leave
 it as is.

  * The 'options' property in Layer.js -- I dont think that guy should be
 public. it's only there for the purpose of being able to clone layers, and
 ideally the user should not touch it. Rather, if the user wishes to add an
 option post-hoc, he/she should use the addOptions() method.

 ... which actually brings to light the idea that the users really
 *shouldn't* be setting *any* of those options directly on the layer but
 should rather be using addOptions(), otherwise their layers will not be
 correctly cloned. But hey. If you just set options to Property instead of
 APIProperty, I think it's good enough.

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


More information about the Trac mailing list