Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Remove inline styles from image captions #7549

Merged
merged 1 commit into from Jan 16, 2015

Conversation

ausi
Copy link
Member

@ausi ausi commented Jan 7, 2015

The inline styles for image captions don’t make sense anymore. Because of the new responsive images feature the image width is dynamic.

Related: #3517, Forum thread 54806

@leofeyer leofeyer added this to the 3.4.1 milestone Jan 10, 2015
@leofeyer
Copy link
Member

Did we not solve this with #7351? And doesn't your change revert the fixes from #3517?

@ausi
Copy link
Member Author

ausi commented Jan 11, 2015

Did we not solve this with #7351?

No, in #7351 we fixed the inconsistency between $this->arrSize / $this->src and $this->picture['img']['width'] / $this->picture['img']['src'].

As commented in #3517 (comment) the inline styles don’t work great for responsive websites. And because of the new responsive images feature the width of the image is dynamic and the caption width may not match the width of the image.

@leofeyer leofeyer merged commit 817e41f into contao:master Jan 16, 2015
@leofeyer
Copy link
Member

Actually merged in 35fe8d9.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants