jQuery 3 compatibility issue

[adamjimenez]

This line breaks jQuery 3 support, seems to work fine without it:
https://github.com/allpro/layout/blob/master/source/stable/jquery.layout.js#L1832

Also .bind / .unbind are deprecated.

[allpro]

The container.selector property was provided only for information. It is never used by Layout so I'll just delete it.

I saw this issue in a forum post. I will be reviewing all the bugs I saw there soon, as well as the pull requests.

[adamjimenez]

Thanks Kevin, it's great that you are back. I was worried that something
had happened to you!
On 3 Jul 2016 2:00 a.m., "Kevin Dalman" notifications@github.com wrote:

The container.selector property was provided only for information. It is
never used by Layout so I'll just delete it.

I saw this issue in a forum post. I will be reviewing all the bugs I saw
there soon, as well as the pull requests.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#10 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/AAi_COzgaHnZEIF7e7w4BfagA0xBMwj9ks5qRwmpgaJpZM4JDuG9
.

[jrgleason]

Any word? We could be a little more backwards compatible perhaps something like sC.selector = $N.selector ? $N.selector.split(".slice")[0] : "";

[allpro]

Backwards compatibility makes sense, but I think undefined is better than an empty string so that it's clear this property is not available with jq3. This would break code that assumes a string value, but if the code relies on this property, it has to be updated anyway. This error would bring attention to the issue rather than create a hard-to-trace bug.

If users really need container object metadata, they can read it directly from layout.container (the jQuery object) instead of layout.state.container.

[marbetschar]

Just stumpled accross this issue. Has it already been addressed?

[marbetschar]

And the following line also breaks in jQuery 3 (please note, I've also have the jQuery migrate plugin installed): https://github.com/allpro/layout/blob/master/source/stable/jquery.layout.js#L2589

Any news on jQuery v3 compatibility?

[melloware]

+1

[fperezgaliana]

Hi,

Is the project still alive?

[melloware]

I use @GedMarc fork of this project which he already implemented Jquery 3. https://github.com/GedMarc/layout

[GedMarc]

I am still maintaining and improving the layout as required, So feel free!

[GedMarc]

No haven't forgotten about you, +- 4 days I think till the new release.

V6 will change the responsive features into a more object orientated feel, and remove the new uniqueSort and binds() deprecation warnings. It also allows elements to be specified as contents for togglers instead of just pure html (with movement not copy).
Specific width settings for responsiveness will also be available to allow for the changes in bootstrap4 regarding sizes.

Please note though that because of a dependency on JQuery UI, so some of these warnings will still pop up until they release a new update.

[melloware]

Fantastic news @GedMarc it is much appreciated!

[djwashburn]

Is there any plan resolve this issue, or should I use the @GedMarc fork mentioned above?

[GedMarc]

This is the stable JQuery 2 Branch

[djwashburn]

Are you saying that this branch is not getting any future changes, and all changes are going to the fork? That is not documented anywhere that I have seen, which is why I am asking. Also, I am not able to create issues in the fork, that I can tell.