Skip to content

Commit

Permalink
Change ExternalLinks icons to use NewWindowIcon component
Browse files Browse the repository at this point in the history
Signed-off-by: Ruben Vargas <ruben.vp8510@gmail.com>
  • Loading branch information
rubenvp8510 committed Dec 10, 2019
1 parent f8d2f44 commit 4c2427f
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions packages/jaeger-ui/src/components/common/ExternalLinks.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,10 @@
import { Dropdown, Icon, Menu } from 'antd';
import * as React from 'react';
import { Link } from '../../types/trace';
import NewWindowIcon from './NewWindowIcon';

type ExternalLinksProps = {
links: Link[];
children?: React.ReactNode;
};

const LinkValue = (props: {
Expand All @@ -34,7 +34,7 @@ const LinkValue = (props: {
rel="noopener noreferrer"
className={props.className}
>
{props.children} <Icon className="KeyValueTable--linkIcon" type="export" />
{props.children} <NewWindowIcon />
</a>
);

Expand All @@ -59,7 +59,7 @@ export default function ExternalLinks(props: ExternalLinksProps) {
return (
<Dropdown overlay={linkValueList(links)} placement="bottomRight" trigger={['click']}>
<a className="TracePageHeader--back">
<Icon className="KeyValueTable--linkIcon is-large" type="profile" />
<NewWindowIcon isLarge={true} />
</a>
</Dropdown>
);
Expand Down

0 comments on commit 4c2427f

Please sign in to comment.