Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

Allow data-class attribute to be copied when replacing background image - #115

Merged
thom4parisot merged 1 commit into
bbc:feature-88from
albertfdp:feature-88-classes-fix
Jan 9, 2015
Merged

Allow data-class attribute to be copied when replacing background image#115
thom4parisot merged 1 commit into
bbc:feature-88from
albertfdp:feature-88-classes-fix

Conversation

@albertfdp

Copy link
Copy Markdown

When a div where the background image is replaced, the existing classes were not kept.

@thom4parisot

Copy link
Copy Markdown
Contributor

Well, the only altered attribute is the style one so I do see why your existing classes are not kept.

Do you rather mean that the gif.className is not added to the div?

@albertfdp

Copy link
Copy Markdown
Author

Sorry, I was not clear: it is just consistency. I am replacing both placeholders and background images on the same page and I think that using the same convention would make sense.

@thom4parisot

Copy link
Copy Markdown
Contributor

Yep you are right, it makes sense :-)

thom4parisot pushed a commit that referenced this pull request Jan 9, 2015
Allow data-class attribute to be copied when replacing background image
@thom4parisot
thom4parisot merged commit 5ae0f6a into bbc:feature-88 Jan 9, 2015
@thom4parisot

Copy link
Copy Markdown
Contributor

Thank you @albertfdp!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants