bootstrap-touch-carousel
bootstrap-touch-carousel copied to clipboard
Do not hide captions on smaller screens
On the example page when viewed on an iPad 2, the content within each ".item" does not show.
(It does swipe, though.)
hey @conoverm, thanks for reporting!
I've conceal the whole captions for smaller screen, cause of the limited space on these devices.
// Hide/rearrange for smaller screens
@media screen and (max-width: 768px) {
// Show and left align the captions
.carousel-caption {
display: none
}
}
However, I could imagine to make this optional. So you can enable captions on smaller screen via data-attributes. would that be an option for you?
Cheers
I've correct the media-query to exclude iPad portrait orientation (max-width:767px).
Have in mind to make this optional.
Cheers
I decided to not include this feature. It feels a bit over-engineered, IMO.
IMO, feels a little over-engineered to hide the captions in the first place. I think the styling details should be left to the implementer.
hey @jtremback, Agreed! will put it into the next release..