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

changed css property #10

Merged
merged 2 commits into from
May 14, 2015
Merged

changed css property #10

merged 2 commits into from
May 14, 2015

Conversation

kambayashia
Copy link

'background:' overwrite many properties.

before

.class-name {
    background: url('data:image/png;.....');
}

after

.class-name {
    background-image: url('data:image/png;.....');
}

@heldr
Copy link
Member

heldr commented May 8, 2015

Hi, could you please update the tests?

https://travis-ci.org/heldr/datauri/jobs/61119118

@kambayashia
Copy link
Author

👌

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 1c929f1 on kambayashia:master into 089ab07 on heldr:master.

@kambayashia
Copy link
Author

updated test.

heldr added a commit that referenced this pull request May 14, 2015
@heldr heldr merged commit 4c18b00 into data-uri:master May 14, 2015
@heldr
Copy link
Member

heldr commented May 14, 2015

Thanks for the contribution 🍵

@kambayashia
Copy link
Author

👍

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

Successfully merging this pull request may close these issues.

None yet

3 participants