Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add responsive CSS to hyperlinked images when thumbnail caption markup used #10662

Merged
merged 1 commit into from
Dec 2, 2013
Merged

add responsive CSS to hyperlinked images when thumbnail caption markup used #10662

merged 1 commit into from
Dec 2, 2013

Conversation

kevinlisota
Copy link
Contributor

When caption markup is used in thumbnails like here http://getbootstrap.com/components/#thumbnails-custom-content, non-hyperlinked images are properly responsive. If you hyperlink the image when using caption markup (like in the default thumbnail example), the image is not responsive because the .thumbnail selector is moved up one level and .thumbnail > img no longer applies to the image.

This fix adds a new selector for responsive images for the case when you wish to use a hyperlinked image followed by a caption.

…p is used

When thumbnail caption markup is used in combination with a hyperlinked image, the image is not responsive. (non-hyperlinked images are fine, like the example in the docs) Adding a selector to apply responsiveness to hyperlinked images when thumbnail captions are used.
@mdo
Copy link
Member

mdo commented Oct 21, 2013

The original intention of the thumbnails for linked images is that the link would be the entire .thumbnail, not within that. However, I can see this being a use case for custom content as you mention. The only downside is that the selector is rather bad—two open ended tags to start it off :. No real way around that as far as I can tell.

I'd really rather not complicate it further, but I'm open to revisiting this in our next-next release (v3.0.2).

mdo added a commit that referenced this pull request Dec 2, 2013
add responsive CSS to hyperlinked images when thumbnail caption markup used
@mdo mdo merged commit cc6951f into twbs:master Dec 2, 2013
@mdo mdo mentioned this pull request Dec 2, 2013
@kevinlisota kevinlisota deleted the thumbnail-images branch December 21, 2013 16:57
stempler pushed a commit to stempler/bootstrap that referenced this pull request Apr 11, 2014
add responsive CSS to hyperlinked images when thumbnail caption markup used
stempler pushed a commit to stempler/bootstrap that referenced this pull request Nov 4, 2014
add responsive CSS to hyperlinked images when thumbnail caption markup used
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants