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

Comments

kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented
Version: 7.x-1.3 » 7.x-1.x-dev
Assigned: regan.dawson@gmail.com » Unassigned
StatusFileSize
new 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
commented
Issue summary: View changes

Fix mistype

manuel garcia’s picture

manuel garcia 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.

astonvictor’s picture

astonvictor commented
Status: Needs review » Closed (outdated)

D7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.