svg-inline.js loads before jquery

  • Resolved waldbach

    (@waldbach)


    11 years, 9 months ago

    Hi,

    Great plugin! But it did not fully work, until I noticed your javascript was not properly placed in the header. It was loaded BEFORE jQuery!

    To fix this, replace:
    wp_register_script('bodhi_svg_inline', plugins_url('svg-support/js/svg-inline.js'), array(''), '1.0.0', false );

    with:

    wp_register_script('bodhi_svg_inline', plugins_url('svg-support/js/svg-inline.js'), array('jquery'), '1.0.0', false );

    in enqueue.php inside functions folder in the plugin directory.

    Cheers! 🙂

    https://ww.wp.xz.cn/plugins/svg-support/

Viewing 15 replies - 1 through 15 (of 24 total)
1 2
  • Plugin Author Benbodhi

    (@benbodhi)

    11 years, 9 months ago

    Hi Waldbach,

    Thanks for your positive feedback and for pointing this issue out!
    What exactly was going wrong for you to notice that?

    Thread Starter waldbach

    (@waldbach)

    11 years, 9 months ago

    Hi Benbodhi,

    It was the fact that <img src="my_vector.svg"> did not convert to a truly embedded SVG. I assume you had that done in the svg-inline.js script, so I checked whether it was loaded correctly and if it depended on jQuery.

    Thread Starter waldbach

    (@waldbach)

    11 years, 9 months ago

    Also, I got the error in the Console of my browser inspector. It said the jQuery variable in your js file was not defined. In most cases this is because it’s placed before the query link as opposed to after it, as it should be.

    Plugin Author Benbodhi

    (@benbodhi)

    11 years, 9 months ago

    Ok, thanks, I will run some tests shortly and update the enqueue.
    It seems to work fine for me for some weird reason… but it certainly should be loaded after jQuery, so I will push out that update in the next few hours.
    Thanks again 🙂

    Thread Starter waldbach

    (@waldbach)

    11 years, 9 months ago

    Cheers, you’re most welcome! Thanks for the great plugin, I am a big fan of SVG! 😀

    Plugin Author Benbodhi

    (@benbodhi)

    11 years, 9 months ago

    I added the jQuery dependency as of version 2.1.6. Update should be available in your admin dashboard now.
    Cheers!

    Thread Starter waldbach

    (@waldbach)

    11 years, 9 months ago

    Sweet, thanks. Hope you did a better job than I did, as for some odd reason it’s back to loading an img tag again.. strange.

    Will check out your update. Cheers!

    Plugin Author Benbodhi

    (@benbodhi)

    11 years, 9 months ago

    It worked for me both before and after adding the jQuery dependency…
    Are you using the correct class to target? By default it is style-svg
    There is detailed usage instructions under Settings > SVG Support in your dashboard (I’m guessing you have checked this out, but you never know, it’s easy to mis-spell a class name or something).

    Thread Starter waldbach

    (@waldbach)

    11 years, 9 months ago

    Yeah, something must’ve gone wrong. It’s a valid SVG file, maybe it is the class indeed, I will check in a moment.
    Right now I am in the process of updating WP to 3.9.2

    Thread Starter waldbach

    (@waldbach)

    11 years, 9 months ago

    oh yes, it must have been the class, i edited the svg positioning from align-left to centered and WP must have regenerated the tag and thus removing the class.

    Would be nice if we could find a way to add (and keep!) this class automatically.

    For the next update I guess!

    Thanks!!

    Plugin Author Benbodhi

    (@benbodhi)

    11 years, 9 months ago

    Let me know how you go, I’d be happy to help out 🙂

    Thread Starter waldbach

    (@waldbach)

    11 years, 9 months ago

    Thanks mate. I’ll keep you posted about bugs and suggestions! 🙂

    Plugin Author Benbodhi

    (@benbodhi)

    11 years, 9 months ago

    I see, the visual editor… That guy does crazy things.
    You’d be better off achieving the desired alignment using CSS in your stylesheet, then it doesn’t matter what the editor does to it.

    Thread Starter waldbach

    (@waldbach)

    11 years, 9 months ago

    Agreed! Was on that already. 🙂

    But it would be great to develop this further (for as far as WP allows us), so any WP end user can just treat it like a regular image. We’ll see, it’s doing great for me as it is!

    Plugin Author Benbodhi

    (@benbodhi)

    11 years, 9 months ago

    Yeah, for sure, that’s a great point!
    I personally don’t use the visual editor, so hadn’t even thought of that.
    I will add it to the future features list. Hopefully I will have time to do it very soon.

Viewing 15 replies - 1 through 15 (of 24 total)
1 2

The topic ‘svg-inline.js loads before jquery’ is closed to new replies.