Add type union for Link target prop (#1643)

Author: brandongregoryscott

index.d.ts

@@ -1398,9 +1398,9 @@ export interface LinkOwnProps extends TextOwnProps {
    */
   href?: string
   /**
-   * Target atrribute, common use case is target="_blank."
+   * Target attribute, common use case is target="_blank."
    */
-  target?: string
+  target?: '_self' | '_blank' | '_parent' | '_top'
   /**
    * The color (and styling) of the Link. Can be default, blue, green or neutral.
    */

index.test-d.ts

@@ -1,5 +1,17 @@
 import { expectAssignable, expectType, expectError } from 'tsd'
-import { defaultTheme, mergeTheme, StyleProps, Intent, Theme, Fill, Partial, Pick, Color, ComboboxProps } from '.'
+import {
+  defaultTheme,
+  mergeTheme,
+  StyleProps,
+  Intent,
+  Theme,
+  Fill,
+  Partial,
+  Pick,
+  Color,
+  ComboboxProps,
+  LinkProps
+} from '.'
 
 interface ThemeOverrides extends Partial<Pick<Theme, 'colors' | 'fills' | 'components'>> {
   colors: {
@@ -131,3 +143,5 @@ expectError(mergeTheme(defaultTheme, themeWithTopLevelColorsArray))
 expectError<ComboboxProps>({ autocompleteProps: { children: 'error' } })
 expectError<ComboboxProps>({ autocompleteProps: { onChange: () => {} } })
 expectError<ComboboxProps>({ autocompleteProps: { items: [] } })
+
+expectError<LinkProps>({ target: 'blank' })

src/typography/src/Link.js

@@ -37,9 +37,9 @@ Link.propTypes = {
   href: PropTypes.string,
 
   /**
-   * Target atrribute, common use case is target="_blank."
+   * Target attribute, common use case is target="_blank."
    */
-  target: PropTypes.string,
+  target: PropTypes.oneOf(['_self', '_blank', '_parent', '_top']),
 
   /**
    * The color (and styling) of the Link. Can be default, blue, green or neutral.