From 4f30dc9e7a2e8e058b101704748eac30b2b423f2 Mon Sep 17 00:00:00 2001 From: "marcins78@gmail.com" Date: Fri, 25 Nov 2011 04:28:08 +0000 Subject: [PATCH] Improving performance of laying out cards. --- .../gemp-lotr-web/src/main/webapp/js/jCardGroup.js | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/gemp-lotr/gemp-lotr-web/src/main/webapp/js/jCardGroup.js b/gemp-lotr/gemp-lotr-web/src/main/webapp/js/jCardGroup.js index 494d13cc4..5b7959412 100644 --- a/gemp-lotr/gemp-lotr-web/src/main/webapp/js/jCardGroup.js +++ b/gemp-lotr/gemp-lotr-web/src/main/webapp/js/jCardGroup.js @@ -335,10 +335,19 @@ var NormalCardGroup = CardGroup.extend({ }); function layoutCardElem(cardElem, x, y, width, height, index) { + x = Math.floor(x); + y = Math.floor(y); + width = Math.floor(width); + height = Math.floor(height); + if (cardElem.css("left") == (x + "px") && cardElem.css("top") == (y + "px") + && cardElem.css("width") == (width + "px") && cardElem.css("height") == (height + "px") + && cardElem.css("zIndex") == index) + return; cardElem.css({position: "absolute", left: x + "px", top: y + "px", width: width, height: height, zIndex: index }); var tokenOverlay = $(".tokenOverlay", cardElem); - tokenOverlay.css({position: "absolute", left: 0 + "px", top: 0 + "px", width: width, height: height}); + if (tokenOverlay.length > 0) + tokenOverlay.css({position: "absolute", left: 0 + "px", top: 0 + "px", width: width, height: height}); $(".foilOverlay", cardElem).css({position: "absolute", left: 0 + "px", top: 0 + "px", width: width, height: height}); @@ -356,7 +365,7 @@ function layoutCardElem(cardElem, x, y, width, height, index) { function layoutTokens(cardElem) { var tokenOverlay = $(".tokenOverlay", cardElem); - if (tokenOverlay != null) { + if (tokenOverlay.length > 0) { var width = cardElem.width(); var height = cardElem.height(); var maxDimension = Math.max(width, height);