-
Notifications
You must be signed in to change notification settings - Fork 55
feat(Icons): Adding chevron down, download, email, search, star icon #955
Conversation
Codecov Report
@@ Coverage Diff @@
## master #955 +/- ##
=========================================
- Coverage 80.72% 80.52% -0.2%
=========================================
Files 659 660 +1
Lines 8465 8453 -12
Branches 1495 1492 -3
=========================================
- Hits 6833 6807 -26
- Misses 1618 1631 +13
- Partials 14 15 +1
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #955 +/- ##
==========================================
+ Coverage 80.5% 80.52% +0.01%
==========================================
Files 659 660 +1
Lines 8450 8453 +3
Branches 1493 1492 -1
==========================================
+ Hits 6803 6807 +4
+ Misses 1632 1631 -1
Partials 15 15
Continue to review full report at Codecov.
|
export default { | ||
icon: ({ classes }) => ( | ||
<svg role="presentation" focusable="false" viewBox="8 8 16 16" className={classes.svg}> | ||
<g> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no harm in leaving it, but the g
isn't necessary
export default { | ||
icon: ({ classes }) => ( | ||
<svg role="presentation" focusable="false" viewBox="8 8 16 16" className={classes.svg}> | ||
<g> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no harm in leaving it, but the g
isn't necessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for all other svg files
…jindal/export-icons
…l/react into pajindal/export-icons
No description provided.