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.