From 2e6e7afe937cc37b43e58b21864675c26a720d2e Mon Sep 17 00:00:00 2001
From: cobbspur <cobbspur@hotmail.com>
Date: Sun, 15 Sep 2013 16:35:53 +0100
Subject: [PATCH] Added hover state for blog-logo and blog-cover using existing
 ids

- blog-logo and blog-cover now have a hover state of cursor: pointer
---
 core/client/tpl/settings/general.hbs | 4 ++--
 core/client/views/settings.js        | 9 ++++++---
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/core/client/tpl/settings/general.hbs b/core/client/tpl/settings/general.hbs
index b5c6921753..2080b712b6 100644
--- a/core/client/tpl/settings/general.hbs
+++ b/core/client/tpl/settings/general.hbs
@@ -22,7 +22,7 @@
             <div class="form-group">
                 <label><strong>Blog Logo</strong></label>
                 {{#if logo}}
-                    <a class="js-modal-logo"><img id="blog-logo" src="{{logo}}" alt="logo"></a>
+                    <a class="js-modal-logo" href="#"><img id="blog-logo" src="{{logo}}" alt="logo"></a>
                 {{else}}
                     <a class="button-add js-modal-logo" href="#">Upload Image</a>
                 {{/if}}
@@ -32,7 +32,7 @@
             <div class="form-group">
                 <label><strong>Blog Cover</strong></label>
                 {{#if cover}}
-                    <a class="js-modal-cover"><img id="blog-cover" src="{{cover}}" alt="cover photo"></a>
+                    <a class="js-modal-cover" href="#"><img id="blog-cover" src="{{cover}}" alt="cover photo"></a>
                 {{else}}
                     <a class="button-add js-modal-cover" href="#">Upload Image</a>
                 {{/if}}
diff --git a/core/client/views/settings.js b/core/client/views/settings.js
index cbe0269cf4..b8263df92d 100644
--- a/core/client/views/settings.js
+++ b/core/client/views/settings.js
@@ -197,11 +197,13 @@
                 this.model.set({availableThemes: themes});
             }
         },
-        showLogo: function () {
+        showLogo: function (e) {
+            e.preventDefault();
             var settings = this.model.toJSON();
             this.showUpload('#logo', 'logo', settings.logo);
         },
-        showCover: function () {
+        showCover: function (e) {
+            e.preventDefault();
             var settings = this.model.toJSON();
             this.showUpload('#cover', 'cover', settings.icon);
         },
@@ -251,7 +253,8 @@
             'click .js-modal-cover': 'showCover',
             'click .js-modal-image': 'showImage'
         },
-        showCover: function () {
+        showCover: function (e) {
+            e.preventDefault();
             var user = this.model.toJSON();
             this.showUpload('#user-cover', 'cover', user.cover);
         },