Ticket #2207 (closed feature: duplicate)

Opened 4 years ago

Last modified 4 years ago

SelectFeature - dont bbox select features if layer is not visible

Reported by: igrcic Owned by: tschaub
Priority: minor Milestone: 2.9 Release
Component: Control.SelectFeature Version: 2.8
Keywords: Cc: ivan.grcic@…
State:

Description

If some of the layers in layers array are not visible, their features get selected anyway

Added patch to skip to next layer if it is no visible. Does something like this has to be added somewhere else, on some other selection?

Attachments

selectFeat_bbox.diff Download (0.6 KB) - added by igrcic 4 years ago.

Change History

Changed 4 years ago by igrcic

Changed 4 years ago by igrcic

I've just seen that this is related to  http://trac.openlayers.org/ticket/1754

Changed 4 years ago by igrcic

Patch should be changed to:

if(!layer.calculateInRange() || !layer.getVisibility()){
    continue;
}

Should we make this optional?

Changed 4 years ago by igrcic

  • cc ivan.grcic, @… added

Changed 4 years ago by igrcic

  • cc ivan.grcic@… added; ivan.grcic, @… removed

Changed 4 years ago by elemoine

Ivan, can we close this ticket and take the discussio

to #1754 ?

Changed 4 years ago by igrcic

Sure, go ahead (sorry I opened ticket before searching old tickets ;)

Changed 4 years ago by elemoine

  • status changed from new to closed
  • resolution set to duplicate
Note: See TracTickets for help on using tickets.