Stylesheet enqueuing Line 113

  • Resolved mcdaniel.isaac

    (@mcdanielisaac)


    11 years, 5 months ago

    Hi I’m looking at the way the stylesheet is enqueued on line 113. In version 4.0.1 it is:

    wp_enqueue_style('mailchimpSF_main_css', home_url('?mcsf_action=main_css&ver='.MCSF_VER));

    Should it actually look like this so that WordPress is aware of the version number for the stylesheet?

    wp_enqueue_style('mailchimpSF_main_css', home_url('?mcsf_action=main_css'), false, MCSF_VER);

    https://ww.wp.xz.cn/plugins/mailchimp/

Viewing 3 replies - 1 through 3 (of 3 total)
  • Thread Starter mcdaniel.isaac

    (@mcdanielisaac)

    11 years, 5 months ago

    Also line 108 should be

    wp_enqueue_style('flick', MCSF_URL.'css/flick/flick.css');

    as this removed the extra slash after MCSF_URL and before css.

    mc_kath

    (@mc_kath)

    11 years, 5 months ago

    Hi mcdaniel.isaac,

    Thanks for posting this! This certainly looks like it would be beneficial to post to the GitHub page for the plugin: https://github.com/crowdfavorite/wp-mailchimp

    Additionally I’ll be happy to pass your notes along to the CrowdFavorite devs directly. They’ve been working on updating the plugin recently and it’s possible your suggestions could be addressed.

    Thanks again!

    -Kath

    Thread Starter mcdaniel.isaac

    (@mcdanielisaac)

    11 years, 5 months ago

    OOOh. Cool Kath thanks for the github link. I’d totally have made posts there or put in a pull request.

    IM

Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Stylesheet enqueuing Line 113’ is closed to new replies.