Skip to content

Commit 9901886

Browse files
Merge pull request #769 from glints-dev/feature/link-monochrome-external
fix: link monochrome with external link
2 parents 6e0b994 + 44c200d commit 9901886

File tree

5 files changed

+27
-1
lines changed

5 files changed

+27
-1
lines changed

src/@next/Link/LinkStyle.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,14 @@ export const StyledLink = styled.a<LinkProps>`
1818
&:hover,
1919
&:active {
2020
color: ${Neutral.B00};
21+
22+
svg {
23+
fill: ${Neutral.B00};
24+
}
25+
}
26+
27+
svg {
28+
fill: ${Neutral.B18};
2129
}
2230
}
2331

test/e2e/link/link.spec.ts

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ test('Link - default', async ({ page }) => {
1313
await expect(linkPage.container).toHaveScreenshot('link-default-pressed.png');
1414
});
1515

16-
test('Link - with extenal link', async ({ page }) => {
16+
test('Link - with external link', async ({ page }) => {
1717
const linkPage = new LinkPage(page);
1818
await linkPage.goto('args=external:true');
1919
await expect(linkPage.container).toHaveScreenshot('link-external.png');
@@ -40,3 +40,21 @@ test('Link - monochrome', async ({ page }) => {
4040
'link-monochrome-pressed.png'
4141
);
4242
});
43+
44+
test('Link - monochrome with external link', async ({ page }) => {
45+
const linkPage = new LinkPage(page);
46+
await linkPage.goto('args=external:true;monochrome:true');
47+
await expect(linkPage.container).toHaveScreenshot(
48+
'link-monochrome-with-external.png'
49+
);
50+
51+
await linkPage.link.hover();
52+
await expect(linkPage.container).toHaveScreenshot(
53+
'link-monochrome-with-external-hover.png'
54+
);
55+
56+
await linkPage.page.mouse.down();
57+
await expect(linkPage.container).toHaveScreenshot(
58+
'link-monochrome-with-external-pressed.png'
59+
);
60+
});
2.43 KB
Loading
2.4 KB
Loading
2.4 KB
Loading

0 commit comments

Comments
 (0)