From 6615f7aad3a51056cdb9d3d323b13c52f177f959 Mon Sep 17 00:00:00 2001
From: Paul Adam Davis <pauladamdavis@gmail.com>
Date: Tue, 5 Aug 2014 18:26:41 +0100
Subject: [PATCH] Swap notification close link for a button

No issue

- Swap out notification <a> close link for <button> (gains a native hover state)
---
 core/client/templates/components/gh-notification.hbs | 2 +-
 core/test/functional/client/editor_test.js           | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/core/client/templates/components/gh-notification.hbs b/core/client/templates/components/gh-notification.hbs
index f2b0a0ba3e..e239c20a04 100644
--- a/core/client/templates/components/gh-notification.hbs
+++ b/core/client/templates/components/gh-notification.hbs
@@ -2,5 +2,5 @@
     <span class="notification-message">
         {{{message.message}}}
     </span>
-    <a class="close" {{action "closeNotification"}}><span class="hidden">Close</span></a>
+    <button class="close" {{action "closeNotification"}}><span class="hidden">Close</span></button>
 </section>
\ No newline at end of file
diff --git a/core/test/functional/client/editor_test.js b/core/test/functional/client/editor_test.js
index 39c2f62907..594c5d3e47 100644
--- a/core/test/functional/client/editor_test.js
+++ b/core/test/functional/client/editor_test.js
@@ -283,7 +283,7 @@ CasperTest.begin('Post settings menu', 30, function suite(test) {
     casper.waitForSelector('.notification-success', function waitForSuccess() {
         test.assert(true, 'got success notification');
         test.assertSelectorHasText('.notification-success', 'Saved.');
-        casper.click('.notification-success a.close');
+        casper.click('.notification-success .close');
     }, function onTimeout() {
         test.assert(false, 'No success notification');
     });
@@ -312,7 +312,7 @@ CasperTest.begin('Post settings menu', 30, function suite(test) {
     casper.waitForSelector('.notification-success', function waitForSuccess() {
         test.assert(true, 'got success notification');
         test.assertSelectorHasText('.notification-success', 'Permalink successfully changed to new-url-editor.');
-        casper.click('.notification-success a.close');
+        casper.click('.notification-success .close');
     }, function onTimeout() {
         test.assert(false, 'No success notification');
     });