Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

Commit a13c651

Browse files
hannakohxtswese44
authored andcommitted
fix(ActionLink): Remove vertical padding from ActionLink (#497)
1 parent ff7e523 commit a13c651

8 files changed

+1
-55
lines changed
Loading
Loading
Loading
Loading
Loading

src/components/ActionLink/ActionLink.styles.ts

Lines changed: 0 additions & 11 deletions
This file was deleted.

src/components/ActionLink/ActionLink.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import Clickable from '../Clickable';
77
import { FixedGridRow, FixedGridColumn, GutterSize } from '../FixedGrid';
88
import { IconSize } from '../Icon';
99
import NavigationLink from '../NavigationLink';
10-
import { getClassNames } from './ActionLink.styles';
1110
import { ActionLinkProps, NavigationActionLinkProps, ClickableActionLinkProps } from './ActionLink.types';
1211

1312
/**
@@ -17,11 +16,9 @@ import { ActionLinkProps, NavigationActionLinkProps, ClickableActionLinkProps }
1716
export default class ActionLink extends React.Component<ActionLinkProps> {
1817
public render() {
1918
const { ariaLabel, className, icon: Icon, text } = this.props;
20-
const classNames = getClassNames();
21-
2219
// Remove Block around Icon when this is addressed: https://github.com/Microsoft/YamUI/issues/327
2320
const content = (
24-
<FixedGridRow gutterSize={GutterSize.SMALL} className={classNames.wrapper}>
21+
<FixedGridRow gutterSize={GutterSize.SMALL}>
2522
<FixedGridColumn fixed={true}>
2623
<Block push={2}>
2724
<Icon size={IconSize.MEDIUM} block={true} />

src/components/ActionLink/__snapshots__/ActionLink.test.tsx.snap

Lines changed: 0 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -8,14 +8,6 @@ exports[`<ActionLink /> with additional className matches its snapshot 1`] = `
88
href="https://yammer.com"
99
>
1010
<FixedGridRow
11-
className=
12-
13-
{
14-
padding-bottom: 0.6rem;
15-
padding-left: 0;
16-
padding-right: 0;
17-
padding-top: 0.2rem;
18-
}
1911
gutterSize="small"
2012
>
2113
<FixedGridColumn
@@ -50,14 +42,6 @@ exports[`<ActionLink /> with href and ariaLabel matches its snapshot 1`] = `
5042
newWindow={true}
5143
>
5244
<FixedGridRow
53-
className=
54-
55-
{
56-
padding-bottom: 0.6rem;
57-
padding-left: 0;
58-
padding-right: 0;
59-
padding-top: 0.2rem;
60-
}
6145
gutterSize="small"
6246
>
6347
<FixedGridColumn
@@ -91,14 +75,6 @@ exports[`<ActionLink /> with href matches its snapshot 1`] = `
9175
newWindow={true}
9276
>
9377
<FixedGridRow
94-
className=
95-
96-
{
97-
padding-bottom: 0.6rem;
98-
padding-left: 0;
99-
padding-right: 0;
100-
padding-top: 0.2rem;
101-
}
10278
gutterSize="small"
10379
>
10480
<FixedGridColumn
@@ -132,14 +108,6 @@ exports[`<ActionLink /> with onClick and ariaLabel matches its snapshot 1`] = `
132108
onClick={[MockFunction]}
133109
>
134110
<FixedGridRow
135-
className=
136-
137-
{
138-
padding-bottom: 0.6rem;
139-
padding-left: 0;
140-
padding-right: 0;
141-
padding-top: 0.2rem;
142-
}
143111
gutterSize="small"
144112
>
145113
<FixedGridColumn
@@ -172,14 +140,6 @@ exports[`<ActionLink /> with onClick matches its snapshot 1`] = `
172140
onClick={[MockFunction]}
173141
>
174142
<FixedGridRow
175-
className=
176-
177-
{
178-
padding-bottom: 0.6rem;
179-
padding-left: 0;
180-
padding-right: 0;
181-
padding-top: 0.2rem;
182-
}
183143
gutterSize="small"
184144
>
185145
<FixedGridColumn

0 commit comments

Comments
 (0)