Fix carousel dots overlapping thumbnails on Product page. #1351
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What?
Things get interesting on the product page image thumbnail carousel when there are a lot of images on that product.
Specifically, when there are more than 22 images the carousel navigation dots will overlap with the image thumbnails.
The way margins are set up, with a window wide enough that Cornerstone will no longer expand out, there will be up to 17 navigation dots per row. These numbers are lower on a narrower view. Using Chrome, this is reduced to 11.
This fix is only for making sure navigation dots don't overlap thumbnail images. The fix also increases the space between the thumbnails and the dots. Only navigation dots on the product page are affected. Carousel navigation on other pages, like the home page, is not.
As an aside, navigation with multiple rows will always seem weird, and the only well-designed solution may be a scroll bar with a variable-width handle, which Slick Carousel (which Cornerstone uses) does not implement.
Tickets / Documentation
Screenshots
Product Pages Before:
Product Pages After:
Narrow Product Page After: