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

Commit

Permalink
style: update network icon colors (#4010)
Browse files Browse the repository at this point in the history
  • Loading branch information
alesmit committed Jul 5, 2021
1 parent c2365c3 commit ab68af0
Show file tree
Hide file tree
Showing 31 changed files with 156 additions and 172 deletions.
Expand Up @@ -25,7 +25,7 @@ exports[`Wallet Card should render 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -439,7 +439,7 @@ exports[`Wallet Card should render with actions 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -584,7 +584,7 @@ exports[`Wallet Card should render with wallet data 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -709,7 +709,7 @@ exports[`Wallet Card should render with wallet data and optional icon 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down
Expand Up @@ -17,7 +17,7 @@ exports[`WalletListItem should render 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV border-theme-primary-100 text-theme-primary-400 group-hover:ring-theme-secondary-100 group-hover:bg-secondary-100 dark:group-hover:ring-black dark:group-hover:bg-black ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 group-hover:ring-theme-secondary-100 group-hover:bg-secondary-100 dark:group-hover:ring-black dark:group-hover:bg-black ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -148,7 +148,7 @@ exports[`WalletListItem should render for selected wallet 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV border-theme-primary-100 text-theme-primary-400 group-hover:ring-theme-secondary-100 group-hover:bg-secondary-100 dark:group-hover:ring-black dark:group-hover:bg-black ring-theme-success-900"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 group-hover:ring-theme-secondary-100 group-hover:bg-secondary-100 dark:group-hover:ring-black dark:group-hover:bg-black ring-theme-success-900"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -283,7 +283,7 @@ exports[`WalletListItem should render with a N/A for fiat 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV border-theme-primary-100 text-theme-primary-400 group-hover:ring-theme-secondary-100 group-hover:bg-secondary-100 dark:group-hover:ring-black dark:group-hover:bg-black ring-theme-success-900"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 group-hover:ring-theme-secondary-100 group-hover:bg-secondary-100 dark:group-hover:ring-black dark:group-hover:bg-black ring-theme-success-900"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -414,7 +414,7 @@ exports[`WalletListItem should render with default BTC as default exchangeCurren
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV border-theme-primary-100 text-theme-primary-400 group-hover:ring-theme-secondary-100 group-hover:bg-secondary-100 dark:group-hover:ring-black dark:group-hover:bg-black ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 group-hover:ring-theme-secondary-100 group-hover:bg-secondary-100 dark:group-hover:ring-black dark:group-hover:bg-black ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down
Expand Up @@ -45,7 +45,7 @@ exports[`ContactListItem should render 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -235,7 +235,7 @@ exports[`ContactListItem should render as delegate 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -439,7 +439,7 @@ exports[`ContactListItem should render using variant 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -622,7 +622,7 @@ exports[`ContactListItem should render with multiple addresses 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -777,7 +777,7 @@ exports[`ContactListItem should render with multiple addresses 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -932,7 +932,7 @@ exports[`ContactListItem should render with multiple addresses 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -1136,7 +1136,7 @@ exports[`ContactListItem should render with multiple options 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -1291,7 +1291,7 @@ exports[`ContactListItem should render with multiple options 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -1446,7 +1446,7 @@ exports[`ContactListItem should render with multiple options 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -1650,7 +1650,7 @@ exports[`ContactListItem should render with one option 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -1790,7 +1790,7 @@ exports[`ContactListItem should render with one option 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -1930,7 +1930,7 @@ exports[`ContactListItem should render with one option 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down
Expand Up @@ -271,7 +271,7 @@ exports[`UpdateContact should render 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 bNDQMV text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down
Expand Up @@ -521,7 +521,7 @@ exports[`Contacts should render with contacts 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -702,7 +702,7 @@ exports[`Contacts should render with contacts 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -871,7 +871,7 @@ exports[`Contacts should render with contacts 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down Expand Up @@ -1040,7 +1040,7 @@ exports[`Contacts should render with contacts 1`] = `
>
<div
aria-label="ARK Devnet"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb border-theme-primary-100 text-theme-primary-400 ring-theme-background"
class="Circle__CircleWrapper-uw7h4j-0 kOlXCb text-theme-secondary-700 border-theme-secondary-300 dark:border-theme-secondary-700 ring-theme-background"
data-testid="NetworkIcon-ARK-ark.devnet"
>
<div
Expand Down
Expand Up @@ -39,7 +39,7 @@ describe("usePortfolioData hook", () => {
const wrapper = ({ children }: any) => <EnvironmentProvider env={env}> {children} </EnvironmentProvider>;
const { result } = renderHook(() => usePortfolioData({ profile }), { wrapper });

expect(result.current.percentages).toEqual([{ color: "danger-400", label: "ARK", percentage: 3 }]);
expect(result.current.percentages).toEqual([{ color: "primary-600", label: "ARK", percentage: 3 }]);

balancePerCoinMock.mockRestore();
});
Expand Down
2 changes: 1 addition & 1 deletion src/domains/dashboard/components/PortfolioChart/hooks.ts
Expand Up @@ -52,7 +52,7 @@ export const usePortfolioData = ({ profile }: { profile: Contracts.IProfile }) =
for (const network of availableNetworks) {
if (network.extra && network.ticker() === coin) {
data.push({
color: network.extra.textClass.replace("text-theme-", ""),
color: "primary-600",
label: coin,
percentage: Number(balancePerCoin[coin].percentage),
});
Expand Down

0 comments on commit ab68af0

Please sign in to comment.