Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

feat(Icon): export ban and exclamation-circle in Teams theme #1123

Merged
merged 5 commits into from
Mar 29, 2019

Conversation

kuzhelov
Copy link
Contributor

@kuzhelov kuzhelov commented Mar 28, 2019

TODO

  • update changelog

@codecov
Copy link

codecov bot commented Mar 28, 2019

Codecov Report

Merging #1123 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1123      +/-   ##
==========================================
+ Coverage   82.03%   82.05%   +0.02%     
==========================================
  Files         722      724       +2     
  Lines        8609     8619      +10     
  Branches     1169     1169              
==========================================
+ Hits         7062     7072      +10     
  Misses       1530     1530              
  Partials       17       17
Impacted Files Coverage Δ
...components/Icon/svg/ProcessedIcons/icons-block.tsx 100% <ø> (ø) ⬆️
...ts/Icon/svg/ProcessedIcons/icons-input-invalid.tsx 100% <ø> (ø) ⬆️
...ms/components/Icon/svg/icons/exclamationCircle.tsx 100% <100%> (ø)
...src/themes/teams/components/Icon/svg/icons/ban.tsx 100% <100%> (ø)
...rc/themes/teams/components/Icon/svg/icons/index.ts 100% <100%> (ø) ⬆️

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 64856ae...8e26115. Read the comment docs.

@@ -17,4 +17,5 @@ export default {
</svg>
),
styles: {},
exportedAs: 'block',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we export as "ban", like font awesome?
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will do that, thanks!

Copy link
Collaborator

@codepretty codepretty left a comment

Choose a reason for hiding this comment

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

Looks good - with one more suggestion... consider exporting the input-invalid to exclamation-circle as well.

@kuzhelov kuzhelov changed the title feat(Icon): export block and input-invalid in Teams theme feat(Icon): export ban and exclamation-circle in Teams theme Mar 29, 2019
@kuzhelov kuzhelov merged commit b45c5de into master Mar 29, 2019
@delete-merged-branch delete-merged-branch bot deleted the feat/export-block-and-input-invalid-icons branch March 29, 2019 01:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants