From a9034de9cd3e376eda95da001532e52293cd83a0 Mon Sep 17 00:00:00 2001
From: Ross Boucher <ross@280north.com>
Date: Mon, 2 Feb 2009 18:16:23 -0700
Subject: [PATCH] Fix for textfield placeholders being completely broken.

---
 AppKit/CPTextField.j | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/AppKit/CPTextField.j b/AppKit/CPTextField.j
index c61740720..17584e3af 100644
--- a/AppKit/CPTextField.j
+++ b/AppKit/CPTextField.j
@@ -389,17 +389,17 @@ var _CPTextFieldSquareBezelColor = nil,
 /* @ignore */
 - (BOOL)acceptsFirstResponder
 {
-    return _isEditable;
+    return _isEditable && _isEnabled;
 }
 
 /* @ignore */
 - (BOOL)becomeFirstResponder
-{
+{    
+#if PLATFORM(DOM)
     var string = [self stringValue];
 
     [self setStringValue:""];
-    
-#if PLATFORM(DOM)
+
     var element = [[self class] _inputElement];
 
     element.value = string;
@@ -479,7 +479,7 @@ var _CPTextFieldSquareBezelColor = nil,
 
     // If current value is the placeholder value, remove it to allow user to update.
     if ([string lowercaseString] == [[self placeholderString] lowercaseString])
-        [self setStringValue:""];
+        element.value = "";
     
     //post CPControlTextDidBeginEditingNotification
     [self textDidBeginEditing:[CPNotification notificationWithName:CPControlTextDidBeginEditingNotification object:self userInfo:nil]];
@@ -519,7 +519,7 @@ var _CPTextFieldSquareBezelColor = nil,
 {
     if (![self isEditable])
         return [[self nextResponder] mouseDown:anEvent];
-        
+
     [super mouseDown:anEvent];
 }
 /*
@@ -683,7 +683,7 @@ var _CPTextFieldSquareBezelColor = nil,
         else
             displayString += aValue;
     }
-    
+
     if ([[self window] firstResponder] == self)
         [[self class] _inputElement].value = displayString;
 
@@ -712,7 +712,7 @@ var _CPTextFieldSquareBezelColor = nil,
 
     //if there is no set value, automatically display the placeholder
     if (!_value) 
-        [self setStringValue:aStringValue];
+        [self setStringValue:_placeholderString];
 }
 
 /*!