On Tuesday, July 21, 2009, Alexandre Dube <ad...@mapgears.com> wrote: > Oh and by the way, here's what I did for the Control (attached). Is it ok > (too much, i.e. about the use of hasOwnProperty functions) ?
why not using the regular var i, len, control; for(i = 0, len = controls.length; i < len; i++) { control = controls[i]; } ? (the OpenLayers way, which is safer than using for in I guess) > > Thanks, > > Alexandre > > Alexandre Dube wrote: > > Eric Lemoine wrote: > > > so, to get to the delete control you've been working on, I see more > value, in terms of flexibility, in adding "control composition" to the > control base class. > > > > > > I also agree. Originally, the control had a Feature handler instead of a > SelectFeature control. Looking at it now, it's too simple to be justified as > a new control. > > > > About the 'controls' property and the act./dea. automatism : that's a good > idea, but wouldn't it be good to have a 'handlers' property also and > activate/deactivate all handlers as well ? Currently, control has a > 'handler' property only. Would it make sense to have 'handlers' added a do > the same kind of automatism as the new 'controls' property ? > > > Agreed. > > > > > Great. > > > > > > > -- > Alexandre Dubé > Mapgears > www.mapgears.com > > -- Eric Lemoine Camptocamp France SAS Savoie Technolac, BP 352 73377 Le Bourget du Lac, Cedex Tel : 00 33 4 79 44 44 96 Mail : eric.lemo...@camptocamp.com http://www.camptocamp.com _______________________________________________ Dev mailing list Dev@openlayers.org http://openlayers.org/mailman/listinfo/dev