Skip to content

Commit

Permalink
make OpenLayers.Feature create OpenLayers.Popup.Anchored as opposed t…
Browse files Browse the repository at this point in the history
…o OpenLayers.Popup.AnchoredBubble by default
  • Loading branch information
elemoine committed Dec 8, 2011
1 parent 33a1c84 commit 46e15b5
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions lib/OpenLayers/Feature.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ OpenLayers.Feature = OpenLayers.Class({
/**
* APIProperty: popupClass
* {<OpenLayers.Class>} The class which will be used to instantiate
* a new Popup. Default is <OpenLayers.Popup.AnchoredBubble>.
* a new Popup. Default is <OpenLayers.Popup.Anchored>.
*/
popupClass: null,

Expand Down Expand Up @@ -188,7 +188,7 @@ OpenLayers.Feature = OpenLayers.Class({
if (!this.popup) {
var anchor = (this.marker) ? this.marker.icon : null;
var popupClass = this.popupClass ?
this.popupClass : OpenLayers.Popup.AnchoredBubble;
this.popupClass : OpenLayers.Popup.Anchored;
this.popup = new popupClass(this.id + "_popup",
this.lonlat,
this.data.popupSize,
Expand Down
2 changes: 1 addition & 1 deletion tests/Feature.html
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
//Safari 3 separates style overflow into overflow-x and overflow-y
var prop = (OpenLayers.BROWSER_NAME == 'safari') ? 'overflowX' : 'overflow';
t.eq(popup.contentDiv.style[prop], "auto", 'overflow on popup is correct');
t.ok( popup instanceof OpenLayers.Popup.AnchoredBubble, "popup is an AnchoredBubble by defaults");
t.ok( popup instanceof OpenLayers.Popup.Anchored, "popup is a Popup.Anchored by default");
feature.destroyPopup();

feature.popupClass = OpenLayers.Popup.FramedCloud;
Expand Down

0 comments on commit 46e15b5

Please sign in to comment.