From d8111bcedb10e3f8676911c292d9a28e52fb0618 Mon Sep 17 00:00:00 2001
From: tidusjar <jamierees@outlook.com>
Date: Wed, 6 Jul 2016 10:03:43 +0100
Subject: [PATCH] Changed the way the donate button works for #414

---
 PlexRequests.UI/Views/Shared/_Layout.cshtml | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/PlexRequests.UI/Views/Shared/_Layout.cshtml b/PlexRequests.UI/Views/Shared/_Layout.cshtml
index 3cc50a93b..80e11ac31 100644
--- a/PlexRequests.UI/Views/Shared/_Layout.cshtml
+++ b/PlexRequests.UI/Views/Shared/_Layout.cshtml
@@ -43,7 +43,7 @@
                     @Html.GetNavbarUrl(Context, "/issues", UI.Layout_Issues, "exclamation", "<span id=\"issueCount\"></span>")
                     @if (Context.CurrentUser.IsAuthenticated()) // TODO replace with IsAdmin
                     {
-                        <li><a id="donate" onclick="donateClick('https://www.paypal.me/PlexRequestsNet');"><i class="fa fa-heart" style="color: red"></i> @UI.Layout_Donate</a></li>
+                        <li><a id="donate"  href="https://www.paypal.me/PlexRequestsNet" target="_blank"><i class="fa fa-heart" style="color: red"></i> @UI.Layout_Donate</a></li>
                     }
                 </ul>
                 <ul class="nav navbar-nav navbar-right">
@@ -112,12 +112,6 @@
 </html>
 <script>
 
-    function donateClick(url) {
-        ga('send', 'event', 'Navbar', 'Donate', 'Donate Clicked');
-        var redirectWindow = window.open(url, '_blank');
-        redirectWindow.location;
-    }
-
     $(function () {
 
         var urlBase = '@Html.GetBaseUrl()';
@@ -179,6 +173,11 @@
         // End issue count
 
 
+        $('#donate').click(function () {
+            ga('send', 'event', 'Navbar', 'Donate', 'Donate Clicked');
+        });
+
+
     });
 
     function scrollToTop() {