Project

General

Profile

Actions

Feature #8972

closed

Accordion Slider plugin

Added by Rowena Li about 7 years ago. Updated about 7 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
-
Target version:
Start date:
2017-12-03
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

Hi

I added Accordion Slider plugin and set it up. It does not respond to click, nor hover. It opens sometimes to one picture. Did I set it up wrong?

Thanks.

https://lbsci720.commons.gc.cuny.edu/

Rowena


Files

accordion.swf (4.54 MB) accordion.swf Rowena Li, 2017-12-03 03:07 PM
Actions #1

Updated by Boone Gorges about 7 years ago

Seems to be working OK for me. Can you share details about your browser/operating system setup? Also, if possible, a screencast showing exactly how it doesn't work would be helpful.

Actions #2

Updated by Rowena Li about 7 years ago

Here is the screencast. Thanks.

Actions #3

Updated by Rowena Li about 7 years ago

Actions #4

Updated by Rowena Li about 7 years ago

I am using Surface Pro Windows 10 in Google Chrome. If I use Microsoft Edge. It is fine. Microsoft Explorer is fine, too. So I guess it is the plugin's problem - not compatible with Google Chrome.

Actions #5

Updated by Boone Gorges about 7 years ago

  • Assignee set to Raymond Hoh

Thanks for the details, Rowena!

It doesn't appear to be a general Chrome issue. The slider appears to be working fine on Chrome on Linux and OSX. Ray, could you test Chrome on Windows to see if you spot an immediate issue? It could be the slider itself, or another script on the page that's killing JS execution. But if it's limited to the Surface Pro environment, our options for fixing are probably limited.

Actions #6

Updated by Raymond Hoh about 7 years ago

I can duplicate on Chromium 64 on Windows.

It appears to be a mixed content issue:

Mixed Content: The page at 'https://lbsci720.commons.gc.cuny.edu/' was loaded over HTTPS, but requested an insecure script 'http://lbsci720.commons.gc.cuny.edu/wp-content/plugins/jquery-vertical-accordion-menu/js/jquery.hoverIntent.minified.js?ver=4.8.4-1.12.3'. This request has been blocked; the content must be served over HTTPS.
(index):1 Mixed Content: The page at 'https://lbsci720.commons.gc.cuny.edu/' was loaded over HTTPS, but requested an insecure script 'http://lbsci720.commons.gc.cuny.edu/wp-content/plugins/jquery-vertical-accordion-menu/js/jquery.cookie.js?ver=4.8.4-1.12.3'. This request has been blocked; the content must be served over HTTPS.
(index):1 Mixed Content: The page at 'https://lbsci720.commons.gc.cuny.edu/' was loaded over HTTPS, but requested an insecure script 'http://lbsci720.commons.gc.cuny.edu/wp-content/plugins/jquery-vertical-accordion-menu/js/jquery.dcjqaccordion.2.9.js?ver=4.8.4-1.12.3'. This request has been blocked; the content must be served over HTTPS.

The jquery-vertical-accordion-menu plugin needs to be patched to work over HTTPS.

The problem appears to be a call to WP_PLUGIN_URL, which I don't believe is HTTPS-compatible:
https://plugins.svn.wordpress.org/jquery-vertical-accordion-menu/trunk/dcwp_jquery_accordion.php

The plugin hasn't been updated in over two years, so maybe we can just patch up the file manually for the upcoming maintenance release.

Actions #7

Updated by Boone Gorges about 7 years ago

  • Target version set to 1.12.4

Good find, Ray! Yeah, I'd say we just patch it on the Commons. If you can find what looks like an official repo on GitHub, feel free to submit a PR there too.

Actions #8

Updated by Raymond Hoh about 7 years ago

  • Status changed from New to Staged for Production Release

Couldn't find an official repo on Github, so I just patched our copy:
https://github.com/cuny-academic-commons/cac/commit/1b7096a8fb647eb75f5173f57c04b6c46fb36348

Actions #9

Updated by Boone Gorges about 7 years ago

  • Status changed from Staged for Production Release to Resolved

Excellent - thanks!

Actions

Also available in: Atom PDF