Just downloaded the latest release and a bunch of the Flexslider Options are missing. These were really useful - why were they removed?

To clarify, I'm talking about the options for -

  • Animation
  • Animation Direction
  • Slide Direction
  • Automatic Playback
  • Slideshow Speed
  • Add Direction Navigation
  • Add Navigation Controls
  • Keyboard Navigation
  • Mousewheel Navigation
  • Pause/Play Controls
  • Randomize Slides
  • Loop
  • Pause on Action
  • Pause on Hover
CommentFileSizeAuthor
#4 add-pauseOnHover-and-conditional-states-1502840-4.patch9.81 KBwillvincent
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

minorOffense’s picture

minorOffense CreditAttribution: minorOffense commented
Status: Active » Closed (works as designed)

Some of these options are specific to the jquery cycle library which aren't supported in flex slider. But others are available in the option set config at admin/config/media/flexslider

Have a look, see if there are options missing on the config page which flex slider supports but I think we got them all. I'll close the issue, if you find anything feel free to open it up again.

Thanks!

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

Ah, that makes sense. I hadn't seen the configuration section yet.

Thanks so much for the quick reply!

Vacilando’s picture

Vacilando CreditAttribution: Vacilando commented

I do not see a setting there for pause on hover (pauseonHover), so at least that one is missing indeed.
Is there any way to add it without hacking the module, e.g. via a variable in settings.php or by override?

willvincent’s picture

willvincent CreditAttribution: willvincent commented
Version: 7.x-1.0-beta2 » 7.x-1.0-rc3
Status: Closed (works as designed) » Needs review
FileSize
add-pauseOnHover-and-conditional-states-1502840-4.patch9.81 KB

Here's a patch against 7.x-1.0-rc3 that adds the pauseOnHover setting, as well as taking care of a couple of the states todos in the option settings form (conditional display of fields based on values of other fields).

This is working for me.

willvincent’s picture

willvincent CreditAttribution: willvincent commented
Title: FlexSlider Options Missing » FlexSlider Pause on Hover option missing

Updating title for clarity

Status: Needs review » Needs work

The last submitted patch, add-pauseOnHover-and-conditional-states-1502840-4.patch, failed testing.

willvincent’s picture

willvincent CreditAttribution: willvincent commented

Patch is not in the format expected by the automated test bot. It does apply cleanly by hand.. this is a p0 format patch, not p1 format.

patch < add-pauseOnHover-and-conditional-states-1502840-4.patch
ANDiTKO’s picture

ANDiTKO CreditAttribution: ANDiTKO commented
Status: Needs work » Fixed

Pause on hover option was added to the 7.x-2.x version.
Just update your FlexSlider module and go to the option set settings (under the Advanced Option tab). Its just below the "Pause On controls" checkbox.

minorOffense’s picture

minorOffense CreditAttribution: minorOffense commented
Version: 7.x-1.0-rc3 » 7.x-2.x-dev
Status: Fixed » Closed (fixed)