[OpenLayers-Dev] DeleteFeature Control

[Christopher Schmidt]

On Tue, Mar 03, 2009 at 08:44:55AM -0500, Alexandre Dube wrote:
> Hi Devs,
> 
> http://trac.openlayers.org/ticket/1882

Alexandre,

Given this, it seems like the current code needs more work. For now,
I've set the ticket to that, and moved it to 2.9, since it seems like we
need discussion, more than code, here. If you expect this to be in 2.8
still, feel free to pull it back.

-- Release Manager

> The way I see this control now the more I think it should have been done 
> an other way.
> 
> Currently :
>   - It uses feature+keyboard+box handlers for selection/unselection
>   - It has its own array of selected features
>   - It directly changes the renderIntent of the selected/unselected 
> features (which is wrong)
> 
> Should instead :
>   - Have an integrated SelectFeature control, like the ModifyFeature 
> control, but with more handlers (keyboard).
> 
> I think this needs discussion :
>   - A SelectFeature control, on deactivation, should always unselect all 
> features.
>   - This control has the following behaviors : keyboards combinations of 
> shift + ctrl for selections and the select on click, more select on 
> click+ctrl and no unselection on click on a selected feature.  They are 
> different than the SelectFeature control.  IMHO, the DeleteFeature 
> behaviors are more standard-like.
> 
> Comments are welcomed.
> 
> -- 
> Alexandre Dubé
> Mapgears
> www.mapgears.com
> 
> _______________________________________________
> Dev mailing list
> Dev at openlayers.org
> http://openlayers.org/mailman/listinfo/dev

-- 
Christopher Schmidt
MetaCarta