From 827bd1f8990e18e1978c95484680d51524ca19bf Mon Sep 17 00:00:00 2001
From: Jay Sitter <jsitter@gmail.com>
Date: Fri, 31 Oct 2014 11:10:38 -0400
Subject: [PATCH] Standardizing class names and styles for close buttons

---
 themes/baggy/_pocheit-form.twig |  2 +-
 themes/baggy/_search-form.twig  |  2 +-
 themes/baggy/css/main.css       | 34 +++++++++++++++------------------
 3 files changed, 17 insertions(+), 21 deletions(-)

diff --git a/themes/baggy/_pocheit-form.twig b/themes/baggy/_pocheit-form.twig
index 409707f09..2068cd31c 100755
--- a/themes/baggy/_pocheit-form.twig
+++ b/themes/baggy/_pocheit-form.twig
@@ -1,7 +1,7 @@
 <div id="bagit-form" class="messages info">
     <form method="get" action="index.php" target="_blank" id="bagit-form-form">
         <h2>{% trans "Save a link" %}</h2>
-        <a href="javascript: void(null);" id="bagit-form-close" class="popup-close">&times;</a>
+        <a href="javascript: void(null);" id="bagit-form-close" class="close-button--popup close-button">&times;</a>
         <input type="hidden" name="autoclose" value="1" />
         <input required placeholder="example.com/article" class="addurl" id="plainurl" name="plainurl" type="url" />
         <span id="add-link-result"></span>
diff --git a/themes/baggy/_search-form.twig b/themes/baggy/_search-form.twig
index 35f3a9947..aec902ff9 100644
--- a/themes/baggy/_search-form.twig
+++ b/themes/baggy/_search-form.twig
@@ -1,6 +1,6 @@
 <div id="search-form" class="messages info">
 <form method="get" action="index.php">
-        <a href="javascript: void(null);" id="search-form-close" class="popup-close">&times;</a>
+        <a href="javascript: void(null);" id="search-form-close" class="close-button--popup close-button">&times;</a>
         <input type="hidden" name="view" value="search"></input>
         <label>{% trans "Search" %}:</label><input required placeholder="{% trans "Enter your search here" %}" type="text" name="search" id="searchfield" />
         <input id="submit-search" type="submit" value="{% trans "Search" %} !"></input>
diff --git a/themes/baggy/css/main.css b/themes/baggy/css/main.css
index 873d17993..017049758 100755
--- a/themes/baggy/css/main.css
+++ b/themes/baggy/css/main.css
@@ -572,24 +572,32 @@ footer a {
 	margin-left: 0;
 }
 
-.popup-close {
+.closeMessage,
+.close-button {
   background: #000;
   color: #FFF;
-	font-size: 1.4em;
-	line-height: 1.6em;
+	font-size: 1.2em;
+	line-height: 1.6;
 	width: 1.6em;
 	height: 1.6em;
 	text-align: center;
   text-decoration: none;
+}
+	.closeMessage:hover,
+	.closeMessage:focus,
+	.close-button:hover,
+	.close-button:focus {
+		background: #999;
+		color: #000;
+	}
+
+.close-button--popup {
 	display: inline-block;
 	position: absolute;
 	top: 0;
 	right: 0;
+	font-size: 1.4em;
 }
-	.popup-close:hover {
-		background: #999;
-		color: #000;
-	}
 
 .active-current {
   background-color: #999;
@@ -787,18 +795,6 @@ margin-top:1em;
 
 .messages > * { display: inline-block;}
 
-.closeMessage {
-  background: #000;
-  color: #FFF;
-  padding: 0.2em 0.5em;
-  text-decoration: none;
-}
-
-  .closeMessage:hover, .closeMessage:focus {
-    background: #FFF;
-    color: #000;
-  }
-
 .warning {
   /* font-size: 3em;
   color: #999;