Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3rd Party Plugin Compatibility: Slider Revolution #614

Closed
moretarman opened this issue Nov 17, 2015 · 28 comments
Closed

3rd Party Plugin Compatibility: Slider Revolution #614

moretarman opened this issue Nov 17, 2015 · 28 comments
Assignees
Milestone

Comments

@moretarman
Copy link

@moretarman moretarman commented Nov 17, 2015

I installed the pro version of Zencache, and enabled the HTML Compression option. All of the pages are properly minimized and compressed on the site, including those making use of Visual Composer.

However, the one page that has the Slider Revolution plugin active refused to compress (HTML/JS/CSS). The unminified page still caches, though.

@jaswrks
Copy link

@jaswrks jaswrks commented Nov 17, 2015

@moretarman Thanks for the report! :-)

You're talking about this plugin right? http://codecanyon.net/item/slider-revolution-responsive-wordpress-plugin/2751380 ~ Is there a free version that we can test against?

@moretarman
Copy link
Author

@moretarman moretarman commented Nov 17, 2015

Yes, that's the one. Let me know if there's anything I can do to help you track down the issue

On Nov 17, 2015, at 12:46 AM, JasWSInc notifications@github.com wrote:

@moretarman Thanks for the report! :-)

You're talking about this plugin right? http://codecanyon.net/item/slider-revolution-responsive-wordpress-plugin/2751380


Reply to this email directly or view it on GitHub.

@raamdev
Copy link
Contributor

@raamdev raamdev commented Nov 17, 2015

@moretarman Is there a free version of the plugin that we can test with?

@moretarman
Copy link
Author

@moretarman moretarman commented Nov 17, 2015

I checked if there was a free version, but I couldn't find one.

@raamdev
Copy link
Contributor

@raamdev raamdev commented Nov 18, 2015

@moretarman Thank you. Unfortunately that makes this difficult to test. I'll leave this GitHub issue open so that we can look into acquiring a copy of that plugin to test against.

@raamdev raamdev changed the title HTML Compression doesn't work when Slider Revolution plugin is present on a page 3rd Party Plugin Compatibility: Slider Revolution Nov 18, 2015
@moretarman
Copy link
Author

@moretarman moretarman commented Nov 18, 2015

I've asked the developers of I can temporarily transfer my license to you, I'll keep you posted

On Nov 17, 2015, at 5:21 PM, Raam Dev notifications@github.com wrote:

@moretarman Thank you. Unfortunately that makes this difficult to test. I'll leave this GitHub issue open so that we can look into acquiring a copy of that plugin to test against.


Reply to this email directly or view it on GitHub.

@moretarman
Copy link
Author

@moretarman moretarman commented Nov 18, 2015

All right, the developers have confirmed that I can send you guys a copy to test locally. Do you have an email address where I can send it?

@raamdev
Copy link
Contributor

@raamdev raamdev commented Nov 18, 2015

@moretarman You can use the following link: http://bit.ly/1OQOJO9 (expires in 24 hours). Please let me know when you've submitted it.

@moretarman
Copy link
Author

@moretarman moretarman commented Nov 19, 2015

I tried to attach the plugin on that page, but I received a "FILE IS TOO BIG" error message. (it's a 5 MB zip)

@jaswrks
Copy link

@jaswrks jaswrks commented Nov 19, 2015

ILE IS TOO BIG" error message. (it's a 5 MB zip)

Very sorry about this. I bumped this limit up to 7MB max so you should be able to do this now. Please try this again when you get a moment. Here is a new link in case the old one expired. http://bit.ly/1PChOwJ


Noting that we can now accept up to 7MB uploads; same as Zendesk limit.

@moretarman
Copy link
Author

@moretarman moretarman commented Nov 19, 2015

All right, got it through!

@raamdev
Copy link
Contributor

@raamdev raamdev commented Nov 19, 2015

@moretarman Can you tell me the ticket number?

@moretarman
Copy link
Author

@moretarman moretarman commented Nov 19, 2015

9590 I think

On Nov 19, 2015, at 3:01 PM, Raam Dev notifications@github.com wrote:

@moretarman Can you tell me the ticket number?


Reply to this email directly or view it on GitHub.

@raamdev
Copy link
Contributor

@raamdev raamdev commented Nov 19, 2015

@moretarman I spent the past hour reviewing the Revslider plugin and testing it against the ZenCache HTML Compressor. I'm not seeing any immediate fix on our end and it's not entirely clear to me why Revslider is breaking the HTML Compressor.

I did find that if I disable the HTML Compressor option to "combine CSS/JS from remote resources" (ZenCache → Plugin Options → HTML Compression → HTML Compression Options → No, do not combine CSS/JS from remote resources), that the page with Revslider compresses just fine, however the browser then reports two JavaScript errors related to the Revslider code. From what I can tell, those errors don't affect the slider itself, as it still appears just fine.

I do see that Revslider fixed an issue with W3TC's Minifying engine back in Version 4.6.5 SkyWood (2nd December 2014):

  • Fixed Compatibility issues with W3C Total Cache Minifying engine

I suspect a similar fix may be necessary for ZenCache's HTML Compression engine. If the developers for Revslider need a copy of ZenCache Pro to get the plugin working with ZenCache, we'd be happy to help them out.

@moretarman
Copy link
Author

@moretarman moretarman commented Nov 20, 2015

I reached out to the Revslider people, and here's what they said:

Thanks for the update. The best thing to do would be to have the ZenCache team contact us directly, and then we can work with them to solve the issue.

So, there you go. It's funny how that one slider breaks the compression, whereas all the others are fine. I bet it has something to do with the strangish XML data that's included on the page.

@raamdev
Copy link
Contributor

@raamdev raamdev commented Nov 20, 2015

@moretarman Do you have an email address or link to a contact page where I can contact the Revslider developers? I did a Google search but didn't find anything. It seems like the only way to reach their support is by purchasing.

@moretarman
Copy link
Author

@moretarman moretarman commented Nov 22, 2015

Their support told me you could email: info@themepunch.com

@raamdev
Copy link
Contributor

@raamdev raamdev commented Nov 22, 2015

Their support told me you could email: info@themepunch.com

I sent them an email and will update here again when I hear back.

@raamdev
Copy link
Contributor

@raamdev raamdev commented Apr 21, 2016

See also: #742

@SkyViewPros
Copy link

@SkyViewPros SkyViewPros commented Jul 21, 2016

Please keep on this issue! I just experienced it with the home page of a site I did for a Marina on Long Island - http://eastshoremarineny.com

The video and entire slider disappears randomly, and re-appears after I clear the cache. What's strange is that it doesn't seem to happen at regular intervals.

@ThePopularizer
Copy link

@ThePopularizer ThePopularizer commented Sep 11, 2016

I'm experiencing an issue with styling with Revolution Slider as well. It seems the inline element styling changes when the body HTML compression is turned on.

@raamdev
Copy link
Contributor

@raamdev raamdev commented Sep 12, 2016

@ThePopularizer Thanks for the extra info! I'm bumping this up a priority and adding this to the milestone that is up for work during the next development cycle (starting later this week).

@raamdev raamdev added this to the Future Release milestone Sep 12, 2016
@jaswrks
Copy link

@jaswrks jaswrks commented Oct 11, 2016

Referencing internal/private ticket:
https://websharks.zendesk.com/agent/tickets/9590

@jaswrks
Copy link

@jaswrks jaswrks commented Oct 11, 2016

Estimating at .25 days for testing. Unknown problem, unknown fix at this time. So estimate on that will need to wait until the underlying problem is identified.

jaswrks pushed a commit to wpsharks/html-compressor that referenced this issue Nov 8, 2016
…full open tag instead of only the CSS or JS URL and content. This allows for exclusion rules that exclude CSS or JS based on an opening tag attribute; e.g., `<style id='rs-plugin-settings-inline-css'`. See also: [this Comet Cache issue](wpsharks/comet-cache#614).
@jaswrks
Copy link

@jaswrks jaswrks commented Nov 8, 2016

Next Release Changelog:

  • HTML Compressor: Updating to the latest release of the HTML Compressor library. Comet Cache now comes with the new improved v161108 (latest release).
  • RevSlider Compat.: This release includes a built-in exclusion rule for the HTML Compressor to allow for improved compatibility with the popular RevSlider plugin for WordPress. The new built-in exclusion rule looks for and automatically bypasses an important style tag that must be preserved for the RevSlider plugin to work in all scenarios; i.e., <style id='rs-plugin-settings-inline-css'.
@renzms
Copy link

@renzms renzms commented Nov 18, 2016

@raamdev @jaswsinc

Confirmed Working

Pages/Posts with Sliders made with Slider Revolution Plugin are cached and compressed properly.

The Sliders also work properly even if CSS/JS is minified with the use of the exclusion rule.

screen shot 2016-11-18 at 8 46 27 pm


However, I did notice that during my testing, the cache file that is served to logged out visitors does not update when editing settings of the slider and/or updating the slides in the Slider on the backend. An old version of the page is served even with Automatic Cache clearing enabled until the cache of the page with the Slider is manually cleared.

@raamdev
Copy link
Contributor

@raamdev raamdev commented Nov 18, 2016

@renzms writes...

However, I did notice that during my testing, the cache file that is served to logged out visitors does not update when editing settings of the slider and/or updating the slides in the Slider on the backend. An old version of the page is served even with Automatic Cache clearing enabled until the cache of the page with the Slider is manually cleared.

Thanks for catching that. It sounds like we need to see if there's something else we can hook into to watch for changes to Slider Revolution. Can you please open a separate GitHub issue for that with steps to reproduce?

@raamdev
Copy link
Contributor

@raamdev raamdev commented Nov 19, 2016

Comet Cache v161119 has been released and includes changes from this GitHub Issue. See the v161119 announcement for further details.


This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#614).

@raamdev raamdev closed this Nov 19, 2016
@wpsharks wpsharks locked and limited conversation to collaborators Nov 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
6 participants