Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated jquery-layout to latest #556

Closed
wants to merge 1 commit into from
Closed

Conversation

geoffd123
Copy link

Using jquery-layout 1.2.0 was giving a 'undefined' error executing cssCur. Updating to 1.3.0 solves this.

@scottburch
Copy link

I did the same change and everything is working lovely with lots of nested dynamic layouts. However, this is also a breaking change. I had to change code working with 1.2.0 to work with 1.3.0.

We may want to let people know about it first.

@gschmidt
Copy link
Contributor

gschmidt commented Jan 9, 2013

Thanks Geoff! If you'd like to be credited in the release notes, could you fill out the Meteor CLA at contribute.meteor.com?

Scott, thanks for calling out that it's a breaking change. We'll make sure to mention that in the release notes.

n1mmy added a commit that referenced this pull request Feb 5, 2013
…t over 1.2.0 despite the RC designation. Fixes #556.
@n1mmy
Copy link
Contributor

n1mmy commented Feb 5, 2013

Merged in 02ae073. Thanks!

For some reason, the patch here didn't apply cleanly to devel, so I downloaded the latest version and dropped it in. 30.79 instead of 30.76.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants