From a3e9fc547cca06da9b0a3449aa447436665b6fb9 Mon Sep 17 00:00:00 2001
From: Mark Otto <markotto@twitter.com>
Date: Sat, 10 Mar 2012 16:15:54 -0800
Subject: clean up search form in navbar, add new variable for placeholder text
 color

---
 less/navbar.less    | 21 +++++++++------------
 less/variables.less |  2 ++
 2 files changed, 11 insertions(+), 12 deletions(-)

(limited to 'less')

diff --git a/less/navbar.less b/less/navbar.less
index 75ad1336d..0a4870ec3 100644
--- a/less/navbar.less
+++ b/less/navbar.less
@@ -127,23 +127,20 @@
     padding: 4px 9px;
     #font > .sans-serif(13px, normal, 1);
     color: @white;
-    color: rgba(255,255,255,.75);
-    background: #666;
-    background: rgba(255,255,255,.3);
-    border: 1px solid #111;
+    background-color: lighten(@navbarBackground, 25%);
+    border: 1px solid darken(@navbarBackground, 15%);
     @shadow: inset 0 1px 2px rgba(0,0,0,.1), 0 1px 0px rgba(255,255,255,.15);
     .box-shadow(@shadow);
     .transition(none);
 
-    // Placeholder text gets special styles; can't be bundled together though for some reason
-    .placeholder(@grayLighter);
-
-    // Hover states
-    &:hover {
-      color: @white;
-      background-color: @grayLight;
-      background-color: rgba(255,255,255,.5);
+    // Placeholder text gets special styles; can't be a grouped selector
+    &:-moz-placeholder {
+      color: @navbarSearchPlaceholderColor;
+    }
+    &::-webkit-input-placeholder {
+      color: @navbarSearchPlaceholderColor;
     }
+
     // Focus states (we use .focused since IE7-8 and down doesn't support :focus)
     &:focus,
     &.focused {
diff --git a/less/variables.less b/less/variables.less
index f3ba7122a..6c251da56 100644
--- a/less/variables.less
+++ b/less/variables.less
@@ -149,6 +149,8 @@
 @navbarLinkBackgroundHover:       transparent;
 @navbarLinkBackgroundActive:      @navbarBackground;
 
+@navbarSearchPlaceholderColor:    #ccc;
+
 
 // Hero unit
 // -------------------------
-- 
cgit v1.2.3