Tribe-events-tooltip break when using Minify scripts and improve caching settin

  • Resolved Hans Schuijff

    (@hanswitteprins)


    4 years, 2 months ago

    Hi Albert,

    In The Events Calendar there is a month view that on desktop has a tooltip on hover showing the title, image and excerpt of an event. After setting up PhastPress that tooltip shows

    =raw title
=dateDisplay
if(imageTooltipSrc.length) {
=title
} if(excerpt.length) {
=raw excerpt
}

    Normally this should be replace by the actual content. It seems this content is retrieved using the WP REST-api with js, but I’m not sure of that fact and found it easier for now to use a workaround. The plugins seems to use a lot of js. I’m using the v2 views of the plugin.

    Trying to disable and enable the script settings, I found the setting to defer scripts isn’t causing this problem, it only occurs when I also enable the setting “Minify scripts and improve caching setting”.

    For now I have disabled that setting and use the wp-rocket settings now to optimize the js-code instead, since I found that it doesn’t disturb TEC’s functionality.

    Is this problem something you want to explore?

    Hans

    The page I need help with: [log in to see the link]

Viewing 8 replies - 1 through 8 (of 8 total)
  • Plugin Author Albert Peschar

    (@kiboit)

    Hey @hanswitteprins,

    I’ve set up a testing install with PhastPress and The Event Calendar and I experience no problems with the tooltip.

    On your site, I don’t see a tooltip. On which element am I supposed to see the tooltip?

    –Albert

    Thread Starter Hans Schuijff

    (@hanswitteprins)

    Hi @kiboit

    Thanks for looking into this. I retested this on a completely fresh WordPress installation (in Local Lightning by Flywheel) and installed only The Events Calendar and PhastPress on it. The test environment was setup using Nginx, PHP v7.3.5 and MySQL v8.0.16

    I left all settings to it’s default values except one: To be able to test this, the option “Use updated calendar designs” in the Display tab of the Events Calendar must be deselected, otherwise you get a month view without tooltips.

    I added one dummy event to test this out with.

    After all that, go to the events calendar in the frontend and use the month view. Hover above the event and you see the problem. You can see a properly functioning tooltip on [ redundant link removed ]

    The new design doesn’t have the problem, since there the tooltip isn’t used anymore. There it just shows daynumbers with dot where the events are and a list below the calendar of the selected date.

    Hope you can recreate with this. For me it isn’t a problem anymore, since wp-rocket complements what doesn’t work in PhastPress. Either way I’m good.

    Thanks,

    Hans

    Plugin Author Albert Peschar

    (@kiboit)

    Hey @hanswitteprins,

    Thanks for the detailed instructions. The bit about “Use updated calendar designs” is what I was missing in my attempts to reproduce the issue.

    I’ve found the source of the problem: certain special characters (so-called control characters) in JavaScript source code were removed by the minification process. This has been fixed in the latest version of PhastPress, 1.67.

    If you like, you can try that version. If not, at least the problem is fixed for other users.

    Thanks for your feedback!

    –Albert

    Thread Starter Hans Schuijff

    (@hanswitteprins)

    Hey @kiboit,

    I retested and can confirm that it works too at my end.

    I also read the latest version supports bunnycdn and tested that again. It now indeed load images and some other files from bunnycdn after I activate that plugin. I still can’t use wp-rocket for the same, since then your plugin seems to be surpassed and the number of requests crows, since the cdn then delivers wp-rockets output.

    In the end test in gtmetrix didn’t seem to show a substantial advantage in pageload speed using bunnycdn. In either case it seems to vary between 2.5 and 3.5 seconds in most tests. So I can only imagine needing it to stressrelieve the server.

    Do you support it now because it should give some extra speed, or is your own conclusion too that it might not make much difference?

    Thanks, Hans

    Plugin Author Albert Peschar

    (@kiboit)

    Hey @hanswitteprins,

    Thanks for testing and confirming that the issue is fixed.

    If you have a fast server, there’s no need to relieve stress on it. ??

    I added support because some users have asked for support for external CDNs. They usually do that because GTmetrix recommends it and doesn’t recognize Cloudflare. But really, I don’t recommend using an external CDN.

    Generally, and if your hosting doesn’t have speed issues, CDNs only help for users that are far away from your server (especially on another continent).

    I usually recommend using Cloudflare instead of a separate CDN, because that doesn’t require an additional connection to be made to the CDN, which slows down page load. Cloudflare is basically a CDN for your entire site. It has the same benefit of speeding up connections for far-away users, but without the additional connection.

    Hope that clarifies things.

    –Albert

    Thread Starter Hans Schuijff

    (@hanswitteprins)

    Thank you. I have Cloudflare setup, perhaps that is why I don’t notice much difference either way. And I’m trying out cloudways hosting with a server nearby my audience, so for most of my customers that should help.

    Cloudways advices to use breeze on their platform and remove other caching plugins. I’ve seen that this plugin does a lot of what you offer too. Do you know that plugin and how does it compare? Should I ignore their advice in this?

    With the config I already have with your plugin and wp-rocket next to it, the numbers are already quit acceptable on desktop. Mobile doesn’t test well yet, but perhaps the theme should deliver less heavy pages on that.

    The ttfb is now an impressive half of what it was (0.12 to 0.15 sec) and it loads pages in about 2.3 seconds (onload is reached in about 1 second), so only the mobile loadtimes are concern at this point.

    In gtmetrix’s waterfall I notice that there seem to be gaps in which no activity happens and there is some relatively large blocking times on files I wasn’t able to explain. I don’t see a relation to other files that are still being loaded. Perhaps I need to study some more to understand it better. ??

    THanks for the explaination and updates.

    Hans

    Plugin Author Albert Peschar

    (@kiboit)

    Hey @hanswitteprins,

    I didn’t know about Breeze before, but by the looks of it is only does caching, minification and compression. PhastPress does a lot more: critical CSS inlining and async script loading. As far as I’m aware, WP Rocket is the only plugin that does this as well (if you enable it).

    PhastPress is not a caching plugin. You should be able to use it with Breeze if you disable their minification and optimization features. I always use WP Super Cache. It shouldn’t make a difference which caching plugin you use, but of course everyone always likes their own plugin best. ??

    PageSpeed mobile scoring depends heavily on CPU usage, which is caused by scripts and complex layouts. The only way to improve that is to simplify pages by removing scripts and unnecessary styling. That’s not something any automated plugin can accomplish. It requires attention from a developer.

    –Albert

    Thread Starter Hans Schuijff

    (@hanswitteprins)

    Well, it works like it is now, so I will leave it alone. ?? WP-Rocket has a good reputation as I understand so it will be fine.

    Thanks.

Viewing 8 replies - 1 through 8 (of 8 total)
  • The topic ‘Tribe-events-tooltip break when using Minify scripts and improve caching settin’ is closed to new replies.