Thanks @baris , seems the same issue indeed. 🙂
UNSOLVED Jquery UI?
-
I was trying to make use of a jqueryui function ( accordion ) in my theme but the console is telling me that it's not a valid function.
Are these not exposed down?
$( function() { $( "#accordion" ).accordion(); } );
-
You need to load it first.
app.loadJQueryUI(function () { $( "#accordion" ).accordion(); });
-
@baris Doh!
-
@baris said in Jquery UI?:
You need to load it first.
app.loadJQueryUI(function () { $( "#accordion" ).accordion(); });
This still fails, I'm a bit perplexed.
Possibly this isn't in the correct place. I'm putting it in as inline javascript in the partial the plugin is using. Should it be elsewhere?
-
@shard I think we use a custom build of jquery ui which doesn't have all the ui widgets. Does others work? For example
.draggable
? -
@baris said in Jquery UI?:
@shard I think we use a custom build of jquery ui which doesn't have all the ui widgets. Does others work? For example
.draggable
?That appears to be the case. I'll open an issue. Should I open a separate one for documenting this custom builds differences?
-
Feel free to PR in the full one. We used a custom build to save on space but since we removed it from nodebb.min.js and introduced that loader method I think it's okay to include all of it
And if you can PR this into our docs that would be great as well
-
@psychobunny said in Jquery UI?:
Feel free to PR in the full one. We used a custom build to save on space but since we removed it from nodebb.min.js and introduced that loader method I think it's okay to include all of it
And if you can PR this into our docs that would be great as well
That might be a bit deeper than I can get. We'll see.
-
Okay. I looked at this and it appears to be a bit more complicated than dropping in a new js.