ambidextrous/gemp-lotr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Improving the client side performance - doing just one layout after all cards are added

Browse Source
This commit is contained in:
marcin.sciesinski
2021-02-22 09:10:16 -08:00
parent 235a76f067
commit 88f8e3bab6
1 changed files with 15 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
gemp-lotr/gemp-lotr-async/src/main/web/js/gemp-022/deckBuildingUi.js
View File

@@ -361,11 +361,12 @@ var GempLotrDeckBuildingUI = Class.extend({
} else if (group == "ring") {
that.addCardToContainer(blueprintId, "special", that.ringDiv, false).addClass("cardInDeck");
} else {
that.addCardToDeckAndLayout(blueprintId, side);
that.addCardToDeckDontLayout(blueprintId, side);
}
}
}
that.layoutSpecialGroups();
that.deckModified(true);
that.layoutDeck();
$("#editingDeck").text("Imported Deck (unsaved)");
});
},
@@ -709,6 +710,18 @@ var GempLotrDeckBuildingUI = Class.extend({
this.selectionFunc = this.addCardToDeckAndLayout;
},
addCardToDeckDontLayout:function (blueprintId, side) {
var that = this;
if (side == "FREE_PEOPLE") {
this.addCardToDeck(blueprintId, side);
} else if (side == "SHADOW") {
this.addCardToDeck(blueprintId, side);
} else if (side == null) {
var div = this.addCardToContainer(blueprintId, side, this.siteDiv, false)
div.addClass("cardInDeck");
}
},
addCardToDeckAndLayout:function (blueprintId, side) {
var that = this;
if (side == "FREE_PEOPLE") {
@@ -948,7 +961,6 @@ var GempLotrDeckBuildingUI = Class.extend({
this.ringBearerGroup.layoutCards();
this.ringGroup.layoutCards();
this.siteGroup.layoutCards();
},
layoutDeck:function () {