sproutcore 1.8.1 → 1.8.2.1

data/lib/frameworks/sproutcore/frameworks/foundation/views/text_field.js

@@ -913,6 +913,9 @@ SC.TextFieldView = SC.FieldView.extend(SC.StaticLayout, SC.Editable,
     // our key/mouse down/up handlers (such as the user choosing Select All
     // from a menu).
     SC.Event.add(input, 'select', this, this._textField_selectionDidChange);
+    
+    // handle a "paste" from app menu and context menu
+    SC.Event.add(input, 'input', this, this._textField_inputDidChange);
   },
 
   /**
@@ -926,6 +929,7 @@ SC.TextFieldView = SC.FieldView.extend(SC.StaticLayout, SC.Editable,
     SC.Event.remove(input, 'blur',   this, this._textField_fieldDidBlur);
     SC.Event.remove(input, 'select', this, this._textField_selectionDidChange);
     SC.Event.remove(input, 'keypress',  this, this._firefox_dispatch_keypress);
+    SC.Event.remove(input, 'input', this, this._textField_inputDidChange);
   },
 
   /** @private
@@ -1004,6 +1008,20 @@ SC.TextFieldView = SC.FieldView.extend(SC.StaticLayout, SC.Editable,
     }
     this.updateHintOnFocus();
   },
+  
+  /** @private
+    Context-menu paste does not trigger fieldValueDidChange normally. To do so, we'll capture the
+    input event and avoid duplicating the "fieldValueDidChange" call if it was already issued elsewhere.
+    
+    I welcome someone else to find a better solution to this problem. However, please make sure that it
+    works with pasting via shortcut, context menu and the application menu on *All Browsers*.
+   */
+  _textField_inputDidChange: function() {
+    var timerNotPending = SC.empty(this._fieldValueDidChangeTimer) || !this._fieldValueDidChangeTimer.get('isValid');
+    if(this.get('applyImmediately') && timerNotPending) {
+      this.invokeLater(this.fieldValueDidChange, 10);
+    }
+  },
 
   /** @private
     Make sure to update visibility of hint if it changes
@@ -1166,12 +1184,10 @@ SC.TextFieldView = SC.FieldView.extend(SC.StaticLayout, SC.Editable,
     }
 
     if (this.get('applyImmediately')) {
-      // There used to be an invokeLater here instead of setTimeout. What we
-      // really need is setTimeout.
-      var self = this;
-      setTimeout(function () {
-        self.fieldValueDidChange();
-      }, 10);
+      // This has gone back and forth several times between invokeLater and setTimeout.
+      // Now we're back to invokeLater, please read the code comment above 
+      // this._textField_inputDidChange before changing it again.
+      this._fieldValueDidChangeTimer = this.invokeLater(this.fieldValueDidChange, 10);
     }
 
     return YES;

data/lib/frameworks/sproutcore/frameworks/runtime/core.js

@@ -59,7 +59,7 @@ window.SproutCore = window.SproutCore || SC ;
 */
 SC = window.SC; // This is dumb but necessary for jsdoc to get it right
 
-SC.VERSION = '1.8.1';
+SC.VERSION = '1.8.2';
 
 /**
   @private

data/lib/frameworks/sproutcore/themes/ace/resources/picker/popover/popover.css

@@ -24,7 +24,7 @@
       left: -25px;
       right: -25px;
       
-      @include slices("popover_notoolbar.png", $left: 40, $top: 40, $bottom: 40, $right: 40, $fill: 1 1, $skip: middle);
+      @include slices("popover_notoolbar.png", $left: 30, $top: 34, $bottom: 34, $right: 30, $fill: 1 1, $skip: middle);
 
       .middle { display: none }
       
@@ -34,24 +34,20 @@
     &.top-toolbar .popover-background {
       top: -20px;
       @include slices(
-        "popover.png", $left: 40, $top: 75, $bottom: 40, $right: 40, 
+        "popover.png", $left: 30, $top: 75, $bottom: 34, $right: 30, 
         $skip: left middle right bottom-left bottom bottom-left bottom-right
       );
-      
-      .left { top: 75px; }
-      .right { top: 75px; }
+
     }
     
     
     &.bottom-toolbar .popover-background {
       bottom: -20px;
       @include slices(
-        "popover.png", $left: 40, $bottom: 75, $top: 40, $right: 40, 
+        "popover.png", $left: 30, $bottom: 75, $top: 34, $right: 30, 
         $skip: left middle right top-left top top-right
       );
       
-      .left { bottom: 75px; }
-      .right { bottom: 75px; }
     }
     
   }
@@ -63,7 +59,7 @@
       top: -25px;
       bottom: -25px;
     
-      @include slices("popover_empty.png", $left: 40, $top: 75, $bottom: 40, $right: 40, $fill: 1 1);
+      @include slices("popover_empty.png", $left: 30, $top: 75, $bottom: 40, $right: 30, $fill: 1 1);
     }
   }
   

metadata

@@ -2,7 +2,7 @@
 name: sproutcore
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 1.8.1
+  version: 1.8.2.1
 platform: ruby
 authors: 
 - Strobe, Inc., Apple Inc. and contributors
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-03-26 00:00:00 Z
+date: 2012-05-13 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: rack