• Resolved ubergiant

    (@ubergiant)


    Great plugin Ronald.

    Issue: It pulls the wrong URL from the home page: digitaldarts.com.au I cannot update the data URL, and it only makes sense to use the url from the actual URL.

    Issue: admin section has miss-spelling “Enter Your Twittter Username”

    Feature Request: A box to type in IDs to exclude pages from using the highlight feature. It’s not good to have the highlight on every page. Some pages you want people focused on a core action rather than sharing.

    https://ww.wp.xz.cn/plugins/highlight-and-share/

Viewing 9 replies - 1 through 9 (of 9 total)
  • Plugin Author Ronald Huereca

    (@ronalfy)

    Thanks for the feedback.

    Unfortunately if we fix it there, it won’t work where it really shines, which is on your blog posts and articles page.

    What I recommend is doing a conditional in functions.php.

    Something like:

    if ( is_front_page() ) {
    	add_filter( 'has_show_facebook', '__return_false' );
    	add_filter( 'has_show_twitter', '__return_false' );
    }
    Plugin Author Ronald Huereca

    (@ronalfy)

    Thanks for the report on the typo. I’ll try to address that this weekend.

    Regarding page IDs, I think most users will not want to manually punch in IDs from my experience.

    A better approach I think is add a small meta box to the page and provide a checkbox that will disable the plugin if checked.

    What do you think?

    Thread Starter ubergiant

    (@ubergiant)

    Can see how that conditional statement would work.

    I want to provide a better suggestion instead of the “actual URL” mentioned earlier. It can continue pulling the URL as it does, but it should first check for twitter card and opengraph markup. That would solve the problem and be more of a best practice.

    Some plugins like GetSocial block themselves from pages that way you suggested. A global setting to allow/disallow on all pages/posts to speed setup would be good and then have the individual settings you speak of that override global.

    Thread Starter ubergiant

    (@ubergiant)

    I have tried:

    if ( is_front_page() {
    	add_filter( 'has_show_facebook', '__return_false' );
    	add_filter( 'has_show_twitter', '__return_false' );
    };

    and as a basic function in functions.php but no change.

    Plugin Author Ronald Huereca

    (@ronalfy)

    What are your Settings-> Reading settings in the admin area? That may provide a clue.

    Thread Starter ubergiant

    (@ubergiant)

    Nothing unusual to me: http://puu.sh/k2PIy/0e0b6605a5.png

    Plugin Author Ronald Huereca

    (@ronalfy)

    Well, posting that screenshot was insanely helpful. It basically said, “I”, as in me, Ronald, screwed up somewhere, and yes, indeed, I did πŸ˜€

    I just released version 1.2.0 which fixes the typo you mentioned above and fixes a bug I wish I had caught way earlier, which is the loading of my filters.

    1. For disabling on the front page, here’s what you can use now: https://github.com/ronalfy/highlight-and-share#disable-sharing-on-a-static-front-page

    2. For changing the content URL, you may now use: https://github.com/ronalfy/highlight-and-share#modify-the-content-url

    Sorry about that and hope the above helps.

    Thread Starter ubergiant

    (@ubergiant)

    Filter works. Thanks.

    Plugin Author Ronald Huereca

    (@ronalfy)

    Very awesome! Support threads like this just help the plugin get more useful.

    Not an obligation by any means, but please consider a rating/review of the plugin.

    Take care!

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

The topic ‘Bugs and Feature Request’ is closed to new replies.