search

geoext / geoext2

GeoExt 2 — JavaScript Toolkit for Rich Web Mapping Applications
http://geoext.github.io/geoext2/
Other
142 stars 106 forks source link

Fixed issue #303 partially #305

Closed KaiVolland closed 9 years ago

KaiVolland commented 9 years ago
  1. Closing the window now deselects the feature.
  2. In ExtJS5 Popup stays in front (alwaysOnTop: true). This is not implemented in ExtJS4.
marcjansen commented 9 years ago

Closing the popup now deselects the feature, but the original issue remains unsolved. I am indifferent on this one. Maybe @chrismayer wants to have a say here as well.

chrismayer commented 9 years ago

Just commented #303 on the general issue.

marcjansen commented 9 years ago

Please have a look at #312.

I think the deselecting behaviour deserves a fix; I'd suggest an upgrade PR once the z.-index stuff is fixed.

marcjansen commented 9 years ago

Can the flesh of this PR ('Closing the popup now deselects the feature') be rebased? I#d consider this an enhancement.

If not doable, please close and open another PR.

marcjansen commented 9 years ago

I am closing this, since it has been quite silent for some time. Please reopen if you disagree.