Skip to content

Conversation

DmitiryPotychkin
Copy link
Contributor

  • Add new icon "Sort"

Screenshot from 2021-02-16 20-27-01

@DmitiryPotychkin DmitiryPotychkin requested a review from a team as a code owner February 16, 2021 18:40
@codecov
Copy link

codecov bot commented Feb 16, 2021

Codecov Report

Merging #601 (525af46) into main (9fa4519) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #601   +/-   ##
=======================================
  Coverage   85.15%   85.15%           
=======================================
  Files         768      768           
  Lines       15862    15862           
  Branches     2037     2037           
=======================================
  Hits        13507    13507           
  Misses       2322     2322           
  Partials       33       33           
Impacted Files Coverage Δ
...ts/assets-library/src/icons/icon-library.module.ts 100.00% <ø> (ø)

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 9fa4519...525af46. Read the comment docs.

@@ -0,0 +1 @@
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" fill="black" width="48px" height="48px"><path d="M0 0h24v24H0z" fill="none"/><path d="M3 18h6v-2H3v2zM3 6v2h18V6H3zm0 7h12v-2H3v2z"/></svg> No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

does it need a fill="currentColor" ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@aaron-steinfeld aaron-steinfeld merged commit 8c1222e into hypertrace:main Feb 18, 2021
@github-actions
Copy link

Unit Test Results

    4 files  ±0  235 suites  ±0   14m 12s ⏱️ +12s
844 tests ±0  844 ✔️ ±0  0 💤 ±0  0 ❌ ±0 
848 runs  ±0  848 ✔️ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 8c1222e. ± Comparison against base commit 9fa4519.

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.

4 participants