Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Issue #12 - fixing TypeError. #13

Merged
merged 1 commit into from

2 participants

@sdafforn
Owner

Underlying problem was to do with an incorrect location value being passed to getRowId. The value was coming from the ContentItemContainer id when it should have been coming from the ciLocation property.

@sdafforn sdafforn Issue #12 - fixing TypeError. Underlying problem was to do with an in…
…correct location value being passed to getRowId. The value was coming from the ContentItemContainer id when it should have been coming from the ciLocation property.
1483240
@cgack cgack merged commit 047816d into master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 4, 2012
  1. @sdafforn

    Issue #12 - fixing TypeError. Underlying problem was to do with an in…

    sdafforn authored
    …correct location value being passed to getRowId. The value was coming from the ContentItemContainer id when it should have been coming from the ciLocation property.
This page is out of date. Refresh to see the latest.
Showing with 4 additions and 4 deletions.
  1. +4 −4 griddle.js
View
8 griddle.js
@@ -563,10 +563,10 @@ $.widget("hx.moveable", {
childs,
childsH = 0,
rowId = -1,
- locId = self.options.ciLocation;
- cln.css("left", "0").css("top", "0").css("position", "relative").addClass("contained");
- locId = $tgt.attr("id").split("ContentItemContainer")[1];
+ location = self.options.ciLocation,
+ locId = $tgt.attr("id").split("ContentItemContainer")[1];
+ cln.css("left", "0").css("top", "0").css("position", "relative").addClass("contained");
cln.data("locationId", locId);
$tgt.children(".sortableregion").append(cln);
$srcEl.remove();
@@ -585,7 +585,7 @@ $.widget("hx.moveable", {
}
$tgt.css("height", childsH + "px");
- rowId = griddle.getRowId($tgt.offset().top, locId);
+ rowId = griddle.getRowId($tgt.offset().top, location);
$(".r" + rowId).css("height", childsH + "px");
griddle.validateLayout(false, "100");
Something went wrong with that request. Please try again.