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

prettify css#3187

Merged
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
jasonLaster:prettier-css
Jun 19, 2017
Merged

prettify css#3187
jasonLaster merged 1 commit into
firefox-devtools:masterfrom
jasonLaster:prettier-css

Conversation

@jasonLaster

@jasonLaster jasonLaster commented Jun 18, 2017

Copy link
Copy Markdown
Contributor

prettier now supports CSS. Lets use it!

@codecov

codecov Bot commented Jun 18, 2017

Copy link
Copy Markdown

Codecov Report

Merging #3187 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3187   +/-   ##
=======================================
  Coverage   47.59%   47.59%           
=======================================
  Files          97       97           
  Lines        4036     4036           
  Branches      828      828           
=======================================
  Hits         1921     1921           
  Misses       2115     2115
Impacted Files Coverage Δ
src/utils/project-search.js 93.75% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a8e5677...1e8588a. Read the comment docs.

@@ -1,5 +1,4 @@
.frames ul .frames-group .group,
.frames ul .frames-group .group .location {
.frames ul .frames-group .group, .frames ul .frames-group .group .location {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! this is a good improvement! but wow that's a complicated selector

@codehag codehag left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great improvement. I think we want to revisit this pr later when we are refactoring css, because there are a few low-hanging fruit (massively long css selectors)

@jasonLaster jasonLaster merged commit 22b7436 into firefox-devtools:master Jun 19, 2017
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