This Week Widget potential issue?

Home Forums Calendar Products Events Calendar PRO This Week Widget potential issue?

Viewing 13 posts - 16 through 28 (of 28 total)
  • Author
    Posts
  • #1036541
    Andy Fragen
    Moderator

    Perhaps stripping the HTML might be an additional thought, at least in the tooltip.

    • This reply was modified 8 years, 5 months ago by Andy Fragen.
    #1036898
    Brian
    Keymaster

    Mark,

    Let me know about the possible work around in this post if that helps you:

    http://theeventscalendar.com/support/forums/topic/this-week-widget-potential-issue/#post-1036513

    Thanks

    #1036935
    Andy Fragen
    Moderator

    Brian,

    Is there a filter for the tooltip text? If not please consider it a feature request. 🙂

    Andy

    #1037180
    Brian
    Keymaster

    There is not a filter. I believe since it runs through the template system it could be replaced for now instead of filtered.

    #1037601
    Steven W
    Participant

    I’m on the road at present and have been limited in what I can do on this, but I did make the code change you recommended and can’t tell that it changed anything. I’ll be back in the office tomorrow (Wednesday) and will test more fully, then let you know. Just didn’t want you to think I’ve dropped out here…

    #1037681
    Brian
    Keymaster

    Sounds good, let us know what you find out tomorrow.

    #1038483
    Steven W
    Participant

    Well, I tried the code change (check http://screencast.com/t/Hhx0lZAuZI to insure I did it correctly) to \wp-content\plugins\the-events-calendar\src\functions\template-tags\general.php and can’t tell that anything changed. I’ve updated to TEC 4.0, TEC Pro 4.0, and TEC Category Colors 4.4.5, but not to WP 4.4 so as not to muddle things (potentially) further. One of Andy’s changes did correct the blank event and I’m now getting the text for the event (i.e., in the Tooltip), but as you can see at http://screencast.com/t/kPNx2BYg4L, I’m still getting the text broken up into lines by punctuation that (significantly?) php recognizes. I also tried changing the ’55’ argument to ‘5’ just in case 55 words was capturing all my fairly short text, but that didn’t change anything either.

    We’ve made progress, and seem to be almost there — but I’ve gone as far as I can go without further direction/suggestion from you or Andy…

    #1038490
    Steven W
    Participant

    BTW, one of the things I did with Andy was to give him the html for one of the events for him to test. He created an all-day event with it and was able to duplicate the problem. If you think that’d be help for you, here’s the event text we’ve been working with:
    <strong>Maundy Thursday<strong> (<span style="color: #800080;">Violet</span> or <span style="color: #ff0000;">Red</span>)</strong></strong> <strong><a href="https://www.biblegateway.com/passage/?search=Exodus+12%3A1-4&version=NIV" target="_blank">Exodus 12:1-4</a></strong>, (<strong><a href="https://www.biblegateway.com/passage/?search=Exodus+12%3A5-10&version=NIV" target="_blank">12:5-10</a></strong>), <strong><a href="https://www.biblegateway.com/passage/?search=Exodus+12%3A11-14&version=NIV" target="_blank">12:11-14</a></strong>; <strong><a href="https://www.biblegateway.com/passage/?search=Psalm+116%3A1-2&version=NIV" target="_blank">Psalm 116:1-2</a></strong>, <strong><a href="https://www.biblegateway.com/passage/?search=Psalm+116%3A12-19&version=NIV" target="_blank">116:12-19</a></strong>; <strong><a href="https://www.biblegateway.com/passage/?search=1+Corinthians+11%3A23-26&version=NIV" target="_blank">1 Corinthians 11:23-26</a></strong>; <strong><a href="https://www.biblegateway.com/passage/?search=John+13%3A1-17&version=NIV" target="_blank">John 13:1-17</a></strong>, <strong><a href="https://www.biblegateway.com/passage/?search=John+13%3A31-35&version=NIV" target="_blank">13:31-35</a></strong>

    #1038510
    Andy Fragen
    Moderator

    Mark,

    I think the solution will be to create an override template for the week view tooltip. I’m looking into it but need to gain a better understanding of the JavaScript templating system.

    #1038528
    Andy Fragen
    Moderator

    OK so here’s a solution.

    https://gist.github.com/afragen/e91341718eb82e52e8eb

    It’s a template override that you need to put into `{your-theme}/tribe-events/pro/week/tooltip.php

    What it does is strip out all the HTML from the tooltip, in the week view only.

    #1038865
    Steven W
    Participant

    OK, that did the trick! After a couple of little styling tweaks, it looks almost exactly as it did before the initial issue appeared. Thanks to both of you for your persistence and your patience with me in working through this. Not sure exactly what it is about you TEC folks (and those of you related to TEC), but I can honestly say that I (who retired from 20 years in IT Management with Nike a few years ago) have never encountered better support on a software issue.

    #1038866
    Steven W
    Participant

    Marking this one resolved.

    #1039061
    Brian
    Keymaster

    Thanks Andy for all your help!

    I am going to go ahead and close this ticket, but if you need help on this or something else please post a new ticket.

    Thanks!

Viewing 13 posts - 16 through 28 (of 28 total)
  • The topic ‘This Week Widget potential issue?’ is closed to new replies.