Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

fix(core): eliminates many (but not all) redundant rules in layout CSS #10509

Merged
merged 1 commit into from
Mar 30, 2017
Merged

fix(core): eliminates many (but not all) redundant rules in layout CSS #10509

merged 1 commit into from
Mar 30, 2017

Conversation

shyndman
Copy link

@shyndman shyndman commented Mar 17, 2017 •
edited
Loading

Pulled the flex-33 and flex-66 classes out of the loop. I also removed some odd duplication in the selector.

Reduction of 179 KB (uncompressed) from core.css.

Fixes #10504.

@googlebot googlebot added the cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ label Mar 17, 2017
@ThomasBurleson
Copy link
Contributor

ThomasBurleson commented Mar 17, 2017 •
edited
Loading

Tested with Docs, Demos, and Karma.
LGTM.

Thx @shyndman
FYI @jelbourn

@ThomasBurleson ThomasBurleson added this to the 1.1.4 milestone Mar 17, 2017
@ThomasBurleson ThomasBurleson added needs: presubmit P1: urgent Urgent issues that should be addressed in the next minor or patch release. labels Mar 17, 2017
@tinayuangao tinayuangao merged commit 90b64fe into master Mar 30, 2017
@GulajavaMinistudio GulajavaMinistudio mentioned this pull request Mar 31, 2017
Merged
@ThomasBurleson ThomasBurleson deleted the fix/10504 branch May 8, 2017 19:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ P1: urgent Urgent issues that should be addressed in the next minor or patch release.
Projects
None yet
Milestone
1.1.4
Development

Successfully merging this pull request may close these issues.
4 participants
@shyndman @ThomasBurleson @googlebot @tinayuangao