-
Notifications
You must be signed in to change notification settings - Fork 193
Conversation
Optimized SVG(s): - src/domains/contacts/images/contacts-banner.svg
@dated failing tests |
@@ -38,7 +38,7 @@ export const Address = ({ address, addressClass, walletName, maxChars, size }: P | |||
addressClass || (walletName ? "text-theme-neutral-400" : "text-theme-neutral-800") | |||
} font-semibold ${size && fontSizes[size]}`} | |||
> | |||
{truncateStringMiddle(address, maxChars)} | |||
{maxChars && maxChars > 0 ? truncateStringMiddle(address, maxChars) : address} |
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.
Should the maxChars && maxChars > 0
part be handled inside truncateStringMiddle
so that it would return the value
without any truncating if it is false? Then it would apply to all places that use truncateStringMiddle
and not just this place.
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.
16e6fad that ok?
Codecov Report
@@ Coverage Diff @@
## 3.0-react #2243 +/- ##
=============================================
Coverage 100.00% 100.00%
=============================================
Files 24 177 +153
Lines 142 1544 +1402
Branches 18 270 +252
=============================================
+ Hits 142 1544 +1402
Continue to review full report at Codecov.
|
Summary
Adds the My Contact page.
Checklist