Skip to content

/ servo

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

style: Stop using PseudoElement::inherits_all. #21946

Merged
merged 2 commits into from Oct 15, 2018
Merged

style: Stop using PseudoElement::inherits_all. #21946

merged 2 commits into from Oct 15, 2018
+37 −42

Commits on Oct 15, 2018

  1. style: Stop using PseudoElement::inherits_all. 

    This was done that way just because Servo didn't support the `all` property at
the time.

We should do it this way and optimize it if it's slow. Though I suspect that
most of stuff doesn't actually need to be inherited, my patch at bug 1498943
should make it much faster than what it would otherwise be.
    @emilio
    emilio committed Oct 15, 2018
    Unverified
    This user has not uploaded their public key yet.
    GPG key ID: 056B727BB9C1027C Learn about signing commits
    7ed3995

  2. style: Remove useless declarations in servo.css. 

    Margin is a reset property, there's no point in setting it to zero since it's
zero by default.
    @emilio
    emilio committed Oct 15, 2018
    Loading status checks…
    Unverified
    This user has not uploaded their public key yet.
    GPG key ID: 056B727BB9C1027C Learn about signing commits
    6d7d1e5
You can’t perform that action at this time.