If you want a margin between your thumbs it will mess up the thumb scroll positioning because the margin isn't taken into account in the calcs.

To fix this I just changed line 22 of infiniteCarousel.js

From: singleWidth = $single.outerWidth(),
To: singleWidth = $single.outerWidth(true),

CommentFileSizeAuthor
#1 2076685-1-singlewidth.patch590 byteskscheirer
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
CreditAttribution: kscheirer commented
Version: 7.x-1.3 » 7.x-1.x-dev
Assigned: regan.dawson@gmail.com » Unassigned
FileSize
2076685-1-singlewidth.patch590 bytes

Converted the above into the attached patch, please test it out!

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
CreditAttribution: kscheirer commented
Issue summary: View changes

Fix mistype

Manuel Garcia’s picture

Manuel Garcia CreditAttribution: Manuel Garcia as a volunteer commented
Issue summary: View changes

Thanks for the patch @kscheirer - it makes sense, but I wonder if would have an effect on the visible calculation that happens next.

I assume it should be ok, but I'm reluctant to get it in because I'm afraid it could break people's sites... so ideally people running this module in production would have to test it and report back if it is safe.