From 1f51ab27c4bcd3fbdaabf96f229dbb995b47a19b Mon Sep 17 00:00:00 2001
From: Brandy Carney <brandyscarney@users.noreply.github.com>
Date: Mon, 1 Jul 2019 11:23:41 -0400
Subject: [PATCH] fix(router-link): add missing target prop (#18659)

references #18655
---
 core/api.txt                                          |  1 +
 core/src/components.d.ts                              |  8 ++++++++
 core/src/components/router-link/readme.md             |  1 +
 core/src/components/router-link/router-link.tsx       | 10 +++++++++-
 core/src/components/router-link/test/basic/index.html |  2 +-
 5 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/core/api.txt b/core/api.txt
index f1073f9232..14cb78051c 100644
--- a/core/api.txt
+++ b/core/api.txt
@@ -930,6 +930,7 @@ ion-router-link,prop,color,string | undefined,undefined,false,false
 ion-router-link,prop,href,string | undefined,undefined,false,false
 ion-router-link,prop,rel,string | undefined,undefined,false,false
 ion-router-link,prop,routerDirection,"back" | "forward" | "root",'forward',false,false
+ion-router-link,prop,target,string | undefined,undefined,false,false
 ion-router-link,css-prop,--background
 ion-router-link,css-prop,--color
 
diff --git a/core/src/components.d.ts b/core/src/components.d.ts
index 0f9d588141..525e3f684b 100644
--- a/core/src/components.d.ts
+++ b/core/src/components.d.ts
@@ -2016,6 +2016,10 @@ export namespace Components {
     * When using a router, it specifies the transition direction when navigating to another page using `href`.
     */
     'routerDirection': RouterDirection;
+    /**
+    * Specifies where to display the linked URL. Only applies when an `href` is provided. Special keywords: `"_blank"`, `"_self"`, `"_parent"`, `"_top"`.
+    */
+    'target': string | undefined;
   }
   interface IonRouterOutlet {
     /**
@@ -5248,6 +5252,10 @@ declare namespace LocalJSX {
     * When using a router, it specifies the transition direction when navigating to another page using `href`.
     */
     'routerDirection'?: RouterDirection;
+    /**
+    * Specifies where to display the linked URL. Only applies when an `href` is provided. Special keywords: `"_blank"`, `"_self"`, `"_parent"`, `"_top"`.
+    */
+    'target'?: string | undefined;
   }
   interface IonRouterOutlet extends JSXBase.HTMLAttributes<HTMLIonRouterOutletElement> {
     /**
diff --git a/core/src/components/router-link/readme.md b/core/src/components/router-link/readme.md
index be83f2fdd5..c65c3af851 100644
--- a/core/src/components/router-link/readme.md
+++ b/core/src/components/router-link/readme.md
@@ -15,6 +15,7 @@ The router link component is used for navigating to a specified link. Similar to
 | `href`            | `href`             | Contains a URL or a URL fragment that the hyperlink points to. If this property is set, an anchor tag will be rendered.                                                                                                                                                | `string \| undefined`           | `undefined` |
 | `rel`             | `rel`              | Specifies the relationship of the target object to the link object. The value is a space-separated list of [link types](https://developer.mozilla.org/en-US/docs/Web/HTML/Link_types).                                                                                 | `string \| undefined`           | `undefined` |
 | `routerDirection` | `router-direction` | When using a router, it specifies the transition direction when navigating to another page using `href`.                                                                                                                                                               | `"back" \| "forward" \| "root"` | `'forward'` |
+| `target`          | `target`           | Specifies where to display the linked URL. Only applies when an `href` is provided. Special keywords: `"_blank"`, `"_self"`, `"_parent"`, `"_top"`.                                                                                                                    | `string \| undefined`           | `undefined` |
 
 
 ## CSS Custom Properties
diff --git a/core/src/components/router-link/router-link.tsx b/core/src/components/router-link/router-link.tsx
index 2b9e224844..66dd51a6df 100644
--- a/core/src/components/router-link/router-link.tsx
+++ b/core/src/components/router-link/router-link.tsx
@@ -36,6 +36,13 @@ export class RouterLink implements ComponentInterface {
    */
   @Prop() routerDirection: RouterDirection = 'forward';
 
+  /**
+   * Specifies where to display the linked URL.
+   * Only applies when an `href` is provided.
+   * Special keywords: `"_blank"`, `"_self"`, `"_parent"`, `"_top"`.
+   */
+  @Prop() target: string | undefined;
+
   private onClick = (ev: Event) => {
     openURL(this.href, ev, this.routerDirection);
   }
@@ -44,7 +51,8 @@ export class RouterLink implements ComponentInterface {
     const mode = getIonMode(this);
     const attrs = {
       href: this.href,
-      rel: this.rel
+      rel: this.rel,
+      target: this.target
     };
     return (
       <Host
diff --git a/core/src/components/router-link/test/basic/index.html b/core/src/components/router-link/test/basic/index.html
index eae3940254..39c2f93d64 100644
--- a/core/src/components/router-link/test/basic/index.html
+++ b/core/src/components/router-link/test/basic/index.html
@@ -30,7 +30,7 @@
       </p>
 
       <p>
-        <ion-router-link href="https://ionicframework.com" rel="external" style="text-decoration: underline">Underline Router Link</ion-router-link>
+        <ion-router-link href="https://ionicframework.com" rel="external" target="_blank" style="text-decoration: underline">External Router Link</ion-router-link>
       </p>
 
       <p>