UNSOLVED Jquery UI?

[Shard]

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();
  } );

[baris]

You need to load it first.

app.loadJQueryUI(function () {
  $( "#accordion" ).accordion();
});

[Shard]

@baris Doh!

[Shard]

@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?

[baris]

@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?

[Shard]

@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?

[psychobunny]

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

[Shard]

@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.

[Shard]

@shard

Okay. I looked at this and it appears to be a bit more complicated than dropping in a new js.