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

merge sequence about "background" #341

Open
sunnylost opened this issue Jun 8, 2017 · 1 comment
Open

merge sequence about "background" #341

sunnylost opened this issue Jun 8, 2017 · 1 comment

Comments

@sunnylost
Copy link

csso version: 3.1.1

I have this css file:

div > a {
    background      : url("about:blank");
    background-size : 100%;
}

.demo > a {
    background-size : 100%;
}

this will generate:

div > a {
    background : url(about:blank)
}

.demo > a, div > a {
    background-size : 100%
}

looks good, but when i swap these tow declarations position,

.demo > a {
    background-size : 100%;
}

div > a {
    background      : url("about:blank");
    background-size : 100%;
}

i got this:

.demo > a, div > a {
    background-size : 100%
}

div > a {
    background : url(about:blank)
}

div > a's background-position would be reseted to its initial value. I know background is a sooooo complicated property, but is this a bug? I have this habit that set background and background-size at the same time, the reason was for compatible with old IE.

@lahmatiy
Copy link
Member

lahmatiy commented Jun 8, 2017

Yep, it's a bug. Duplicates #143

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

No branches or pull requests

2 participants