Skip to content

Commit

Permalink
Removing the tileAnimation map property.
Browse files Browse the repository at this point in the history
Edit CSS to change the style.
  • Loading branch information
tschaub committed Jan 2, 2012
1 parent 78963a5 commit 3c910c8
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 31 deletions.
11 changes: 0 additions & 11 deletions lib/OpenLayers/Map.js
Original file line number Diff line number Diff line change
Expand Up @@ -395,14 +395,6 @@ OpenLayers.Map = OpenLayers.Class({
* the popup from getting too close to the map border.
*/
paddingForPopups : null,

/**
* APIProperty: tileAnimation
* {Boolean} If set to true the olMapTileAnim class is assigned to the
* map div. This, with OpenLayers' default CSS, results in fade-in
* effects when image tiles are displayed.
*/
tileAnimation: true,

/**
* Property: minPx
Expand Down Expand Up @@ -508,9 +500,6 @@ OpenLayers.Map = OpenLayers.Class({
}

OpenLayers.Element.addClass(this.div, 'olMap');
if (this.tileAnimation) {
OpenLayers.Element.addClass(this.div, 'olMapTileAnim');
}

// the viewPortDiv is the outermost div we modify
var id = this.id + "_OpenLayers_ViewPort";
Expand Down
2 changes: 1 addition & 1 deletion tests/Layer/Image.html
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@
function test_loadEvents(t) {
t.plan(3);

var map = new OpenLayers.Map('map', {tileAnimation: false});
var map = new OpenLayers.Map('map');
var layer = new OpenLayers.Layer.Image(
'Test', '../../img/blank.gif',
new OpenLayers.Bounds(-180, -88.759, 180, 88.759),
Expand Down
18 changes: 0 additions & 18 deletions tests/Map.html
Original file line number Diff line number Diff line change
Expand Up @@ -115,24 +115,6 @@
map.destroy();
}

function test_Map_constructor_tileanim(t) {
t.plan(4);

map = new OpenLayers.Map("map");
t.eq(map.tileAnimation, true,
'tileAnimation is true by default');
t.ok(OpenLayers.Element.hasClass(map.div, 'olMapTileAnim'),
'map div is assigned the olMapTileAnim class by default');
map.destroy();

map = new OpenLayers.Map("map", {tileAnimation: false});
t.eq(map.tileAnimation, false,
'tileAnimation is false if option is false');
t.ok(OpenLayers.Element.hasClass(map.div, 'olMapTileAnim'),
'map div not assigned the olMapTileAnim class if tileAnimation is false');
map.destroy();
}

function test_Map_setOptions(t) {
t.plan(2);
map = new OpenLayers.Map('map', {maxExtent: new OpenLayers.Bounds(100, 200, 300, 400)});
Expand Down
2 changes: 1 addition & 1 deletion theme/default/style.css
Original file line number Diff line number Diff line change
Expand Up @@ -433,7 +433,7 @@ span.olGoogleAttribution.hybrid a, span.olGoogleAttribution.satellite a {
* Animations
*/

.olMapTileAnim .olTileImage {
.olTileImage {
-webkit-transition: opacity 0.2s linear;
-moz-transition: opacity 0.2s linear;
-o-transition: opacity 0.2s linear;
Expand Down

0 comments on commit 3c910c8

Please sign in to comment.