sproutcore 1.0.1003 → 1.0.1008

data/frameworks/sproutcore/frameworks/desktop/views/scroll.js

@@ -59,6 +59,12 @@ SC.ScrollView = SC.View.extend(SC.Border, {
     if (!this.get('canScrollHorizontal')) return 0 ;
     var view = this.get('contentView') ;
     var contentWidth = view ? view.get('frame').width : 0 ;
+    
+    // The following code checks if there is a calculatedWidth (collections)
+    // to avoid looking at the incorrect value calculated by frame.
+    if(view.calculatedWidth && view.calculatedWidth!==0){
+      contentWidth = view.calculatedWidth; 
+    }
     var containerWidth = this.get('containerView').get('frame').width ;
     return Math.max(0, contentWidth-containerWidth) ;
   }.property(),
@@ -73,7 +79,13 @@ SC.ScrollView = SC.View.extend(SC.Border, {
   maximumVerticalScrollOffset: function() {
     if (!this.get('canScrollVertical')) return 0 ;
     var view = this.get('contentView') ;
-    var contentHeight = view ? view.get('frame').height : 0 ;
+    var contentHeight = (view && view.get('frame')) ? view.get('frame').height : 0 ;
+    
+    // The following code checks if there is a calculatedWidth (collections)
+    // to avoid looking at the incorrect value calculated by frame.
+    if(view.calculatedHeight && view.calculatedHeight!==0){
+      contentHeight = view.calculatedHeight; 
+    }
     var containerHeight = this.get('containerView').get('frame').height ;
     return Math.max(0, contentHeight-containerHeight) ;
   }.property(),
@@ -302,28 +314,33 @@ SC.ScrollView = SC.View.extend(SC.Border, {
     the view should be a subview of the contentView.  Otherwise the results
     will be undefined.
     
-    @param {SC.ScrollView} receiver
+    @param {SC.View} view view to scroll or null to scroll receiver visible
+    @returns {Boolean} YES if scroll position was changed
   */
   scrollToVisible: function(view) {
-    var contentView = this.get('contentView') ;
-    if (!contentView) return this; // nothing to do if no contentView.
     
-    // get the viewportOffset for the view layer the convert that.  this will
-    // work even  with views using static layout.
-    var layer = view.get('layer'), vf ;
-    if(!layer) return this ; // nothing to do
-    vf =  SC.viewportOffset(layer) ;
-    vf.width = layer.offsetWidth ;
-    vf.height = layer.offsetHeight ;
+    // if no view is passed, do default
+    if (arguments.length === 0) return sc_super(); 
+    
+    var contentView = this.get('contentView') ;
+    if (!contentView) return NO; // nothing to do if no contentView.
+
+    // get the frame for the view - should work even for views with static 
+    // layout, assuming it has been added to the screen.
+    var vf = view.get('frame');
+    if (!vf) return NO; // nothing to do
     
     // convert view's frame to an offset from the contentView origin.  This
     // will become the new scroll offset after some adjustment.
-    vf = contentView.convertFrameFromView(vf, null) ;
+    vf = contentView.convertFrameFromView(vf, view.get('parentView')) ;
     
     // find current visible frame.
     var vo = SC.cloneRect(this.get('containerView').get('frame')) ;
+    
     vo.x = this.get('horizontalScrollOffset') ;
     vo.y = this.get('verticalScrollOffset') ;
+
+    var origX = vo.x, origY = vo.y;
     
     // if top edge is not visible, shift origin
     vo.y -= Math.max(0, SC.minY(vo) - SC.minY(vf)) ;
@@ -334,7 +351,10 @@ SC.ScrollView = SC.View.extend(SC.Border, {
     vo.x += Math.max(0, SC.maxX(vf) - SC.maxX(vo)) ;
     
     // scroll to that origin.
-    return this.scrollTo(vo.x, vo.y) ;
+    if ((origX !== vo.x) || (origY !== vo.y)) {
+      this.scrollTo(vo.x, vo.y);
+      return YES ;
+    } else return NO;
   },
   
   /**
@@ -525,10 +545,8 @@ SC.ScrollView = SC.View.extend(SC.Border, {
     in the regular properties.
   */
   createChildViews: function() {
-    // debugger ;
-    var childViews = [] ;
-    var view ;
-    
+    var childViews = [] , view; 
+       
     // create the containerView.  We must always have a container view. 
     // also, setup the contentView as the child of the containerView...
     if (SC.none(view = this.containerView)) view = SC.ContainerView;
@@ -544,7 +562,8 @@ SC.ScrollView = SC.View.extend(SC.Border, {
     if (view=this.horizontalScrollerView) {
       if (this.get('hasHorizontalScroller')) {
         view = this.horizontalScrollerView = this.createChildView(view, {
-          layoutDirection: SC.LAYOUT_HORIZONTAL
+          layoutDirection: SC.LAYOUT_HORIZONTAL,
+          valueBinding: '*owner.horizontalScrollOffset'
         }) ;
         childViews.push(view);
       } else this.horizontalScrollerView = null ;
@@ -554,7 +573,8 @@ SC.ScrollView = SC.View.extend(SC.Border, {
     if (view=this.verticalScrollerView) {
       if (this.get('hasVerticalScroller')) {
         view = this.verticalScrollerView = this.createChildView(view, {
-          layoutDirection: SC.LAYOUT_VERTICAL
+          layoutDirection: SC.LAYOUT_VERTICAL,
+          valueBinding: '*owner.verticalScrollOffset'
         }) ;
         childViews.push(view);
       } else this.verticalScrollerView = null ;
@@ -563,7 +583,7 @@ SC.ScrollView = SC.View.extend(SC.Border, {
     // set childViews array.
     this.childViews = childViews ;
     
-    this.contentViewFrameDidChange() ; // setup initial display...
+    this.contentViewDidChange() ; // setup initial display...
     this.tile() ; // set up initial tiling
   },
   
@@ -606,7 +626,7 @@ SC.ScrollView = SC.View.extend(SC.Border, {
       if (newView) newView.addObserver('frame', this, f);
       
       // replace container
-      this.containerView.set('content', newView);
+      this.containerView.set('contentView', newView);
       
       this.contentViewFrameDidChange();
     }
@@ -618,7 +638,12 @@ SC.ScrollView = SC.View.extend(SC.Border, {
     size of the contentView changes.  We don't care about the origin since
     that is tracked separately from the offset values.
   */
+  
+  oldMaxHOffset: 0,
+  oldMaxVOffset: 0,
+  
   contentViewFrameDidChange: function() {
+        
     var view   = this.get('contentView'), 
         f      = (view) ? view.get('frame') : null,
         width  = (f) ? f.width : 0,  
@@ -626,7 +651,7 @@ SC.ScrollView = SC.View.extend(SC.Border, {
         dim    = this.get('frame') ;
     
     // cache out scroll settings...
-    if ((width === this._scroll_contentWidth) && (height === this._scroll_contentHeight)) return ;
+    //if ((width === this._scroll_contentWidth) && (height === this._scroll_contentHeight)) return ;
     this._scroll_contentWidth = width;
     this._scroll_contentHeight = height ;
     
@@ -648,6 +673,33 @@ SC.ScrollView = SC.View.extend(SC.Border, {
       height -= this.get('verticalScrollerBottom') ;
       view.setIfChanged('maximum', height) ;
     }
+    
+    // If there is no vertical scroller and auto hiding is on, make
+    // sure we are at the top if not already there
+    if (!this.get('isVerticalScrollerVisible') && (this.get('verticalScrollOffset') !== 0) && 
+       this.get('autohidesVerticalScroller')) {
+      this.set('verticalScrollOffset', 0);
+    }
+    
+    // Same thing for horizontal scrolling.
+    if (!this.get('isHorizontalScrollerVisible') && (this.get('horizontalScrollOffset') !== 0) && 
+       this.get('autohidesHorizontalScroller')) {
+      this.set('horizontalScrollOffset', 0);
+    }
+    
+    // This forces to recalculate the height of the frame when is at the bottom
+    // of the scroll and the content dimension are smaller that the previous one
+    
+    
+    var mxVOffSet = this.get('maximumVerticalScrollOffset'),
+        vOffSet = this.get('verticalScrollOffset'),
+        mxHOffSet = this.get('maximumHorizontalScrollOffset'),
+        hOffSet = this.get('horizontalScrollOffset');
+    var forceHeight = mxVOffSet && this.get('hasVerticalScroller') && mxVOffSet<vOffSet;
+    var forceWidth = mxHOffSet && this.get('hasHorizontalScroller') && mxHOffSet<hOffSet;
+    if(forceHeight || forceWidth){
+      this.forceDimensionsRecalculation(forceWidth, forceHeight, vOffSet, hOffSet);
+    }
   },
   
   /** @private
@@ -657,15 +709,12 @@ SC.ScrollView = SC.View.extend(SC.Border, {
   _scroll_horizontalScrollOffsetDidChange: function() {
     var offset = this.get('horizontalScrollOffset');
     
+    offset = Math.max(0,Math.min(this.get('maximumHorizontalScrollOffset'), offset)) ;
+    
     // update the offset for the contentView...
     var contentView = this.get('contentView');
     if (contentView) contentView.adjust('left', 0-offset);
-    
-    // update the value of the horizontal scroller...
-    var scroller ;
-    if (this.get('hasHorizontalScroller') && (scroller=this.get('horizontalScrollerView'))) {
-      scroller.set('value', offset);
-    }
+     
   }.observes('horizontalScrollOffset'),
   
   /** @private
@@ -675,15 +724,32 @@ SC.ScrollView = SC.View.extend(SC.Border, {
   _scroll_verticalScrollOffsetDidChange: function() {
     var offset = this.get('verticalScrollOffset') ;
     
+    offset = Math.max(0,Math.min(this.get('maximumVerticalScrollOffset'), offset)) ;
+    
     // update the offset for the contentView...
     var contentView = this.get('contentView');
+    var containerView = this.get('containerView');
+    
+    // Optimization when not using collections. We need to reimplement clippingFrame
+    // and scrolling to be able to scroll using scrolltop. For now I just
+    // detect if the content to scroll is a class of collectionView.
     if (contentView) contentView.adjust('top', 0-offset) ;
     
-    // update the value of the vertical scroller...
-    var scroller;
-    if (this.get('hasVerticalScroller') && (scroller=this.get('verticalScrollerView'))) {
-      scroller.set('value', offset) ;
+  }.observes('verticalScrollOffset'),
+  
+  forceDimensionsRecalculation: function (forceWidth, forceHeight, vOffSet, hOffSet) {
+    var oldScrollHOffset = hOffSet;
+    var oldScrollVOffset = vOffSet;
+    this.scrollTo(0,0);
+    if(forceWidth && forceHeight){
+      this.scrollTo(this.get('maximumHorizontalScrollOffset'), this.get('maximumVerticalScrollOffset'));
+    }
+    if(forceWidth && !forceHeight){
+      this.scrollTo(this.get('maximumHorizontalScrollOffset'), oldScrollVOffset);
+    }
+    if(!forceWidth && forceHeight){
+      this.scrollTo(oldScrollHOffset ,this.get('maximumVerticalScrollOffset'));
     }
-  }.observes('verticalScrollOffset')
+  }
   
 });

data/frameworks/sproutcore/frameworks/desktop/views/scroller.js

@@ -40,7 +40,9 @@ SC.ScrollerView = SC.View.extend({
     if (val !== undefined) {
       // Don't enforce the maximum now, because the scroll view could change
       // height and we want our content to stay put when it does.
-      this._value = val ;
+      if (val >= 0) {
+        this._value = val ;
+      }
     } else {
       var value = this._value || 0 ; // default value is at top/left
       return Math.min(value, this.get('maximum')) ;
@@ -216,12 +218,6 @@ SC.ScrollerView = SC.View.extend({
         }
       }
     }
-    
-    // notify owner if it has a different scroll value
-    var key = this.get('ownerScrollValueKey') ;
-    if (key && this.owner && (this.owner[key] !== undefined)) {
-      this.owner.setIfChanged(key, v) ;
-    }
   }.observes('value')
   
 });

data/frameworks/sproutcore/frameworks/desktop/views/segmented.js

@@ -430,13 +430,6 @@ SC.SegmentedView = SC.View.extend(SC.Control,
       this._isMouseDown = YES ;
       this.set('activeIndex', idx);
     }
-    if (!this._isFocused) {
-      this._isFocused = YES ;
-      this.becomeFirstResponder();
-      if (this.get('isVisibleInWindow')) {
-        this.$()[0].focus();
-      }
-    }
     
     return YES ;
   },

data/frameworks/sproutcore/frameworks/desktop/views/separator.js

@@ -27,9 +27,8 @@ SC.SeparatorView = SC.View.extend(
   layoutDirection: SC.LAYOUT_HORIZONTAL,
 
   render: function(context, firstTime) {
-
-    context.push('<span></span>');
-	context.addClass(this.get('layoutDirection'));
+    if(firstTime) context.push('<span></span>');
+	  context.addClass(this.get('layoutDirection'));
   }
 
 

data/frameworks/sproutcore/frameworks/desktop/views/slider.js

@@ -120,16 +120,8 @@ SC.SliderView = SC.View.extend(SC.Control,
   
   mouseDown: function(evt) {
     if (!this.get('isEnabled')) return YES; // nothing to do...
-    
     this.set('isActive', YES);
     this._isMouseDown = YES ;
-    if (!this._isFocused) {
-      this._isFocused = YES ;
-      this.becomeFirstResponder();
-      if (this.get('isVisibleInWindow')) {
-        this.$()[0].focus();
-      }
-    }
     return this._triggerHandle(evt);
   },
   

data/frameworks/sproutcore/frameworks/desktop/views/source_list_group.js

@@ -89,7 +89,7 @@ SC.SourceListGroupView = SC.View.extend(SC.Control, SC.CollectionGroup,
     var labelView = this.outlet('labelView') ;
     
     // hide labelView if content is null.
-    if (content == null) {
+    if (content === null) {
       labelView.setIfChanged('isVisible', NO) ;
       this.setIfChanged('hasGroupTitle', NO) ;
       return ;

data/frameworks/sproutcore/frameworks/desktop/views/split.js

@@ -65,8 +65,7 @@ SC.RESIZE_BOTTOM_RIGHT = 'resize-bottom-right' ;
   viewhelper.
   
   @property {Boolean} canCollapseViews Set to NO when you don't want any of
-  the child views to collapse. Defaults to YES. Use the :can_collapse_views
-  option with the split_view viewhelper.
+  the child views to collapse. Defaults to YES. 
   
   In addition, the top/left and bottom/right child views can have these
   properties:
@@ -383,24 +382,40 @@ SC.SplitView = SC.View.extend(
   renderLayout: function(context, firstTime) {
     // console.log('%@.renderLayout(%@, %@)'.fmt(this, context, firstTime));
     // console.log('%@.frame = %@'.fmt(this, SC.inspect(this.get('frame'))));
-    if (firstTime) {
+    if (firstTime || this._recalculateDivider) {
       if (!this.get('thumbViewCursor')) {
         this.set('thumbViewCursor', SC.Cursor.create()) ;
       }
       
       var layoutDirection = this.get('layoutDirection') ;
-      var splitViewThickness = (layoutDirection == SC.LAYOUT_HORIZONTAL) ? this.get('frame').width : this.get('frame').height ;
+      var fr = this.get('frame');
+      var splitViewThickness, elemRendered = this.$();
+      var dividerThickness = this.get('dividerThickness') || 7 ;
       var desiredThickness = this.get('defaultThickness') ;
       var autoResizeBehavior = this.get('autoresizeBehavior') ;
+      // Turn a flag on to recalculate the spliting if the desired thickness
+      // is a percentage
+      // debugger;
+      if(this._recalculateDivider===undefined && desiredThickness<1) {
+        this._recalculateDivider=YES;
+      }
+      else if(this._recalculateDivider) this._recalculateDivider=NO;
+      
       
+      if(elemRendered[0]) {
+        splitViewThickness = (layoutDirection == SC.LAYOUT_HORIZONTAL) ? 
+              elemRendered[0].offsetWidth : elemRendered[0].offsetHeight ;
+      }else{
+        splitViewThickness = (layoutDirection == SC.LAYOUT_HORIZONTAL) ? 
+              fr.width : fr.height ;
+      }
       // if default thickness is < 1, convert from percentage to absolute
       if (SC.none(desiredThickness) || (desiredThickness > 0 && desiredThickness < 1)) {
-        desiredThickness =  Math.floor(splitViewThickness * (desiredThickness || 0.5)) ;
+        desiredThickness =  Math.floor((splitViewThickness - (dividerThickness))* (desiredThickness || 0.5)) ;
       }
       if (autoResizeBehavior === SC.RESIZE_BOTTOM_RIGHT) {
         this._desiredTopLeftThickness = desiredThickness ;
       } else { // (autoResizeBehavior === SC.RESIZE_TOP_LEFT)
-        var dividerThickness = this.get('dividerThickness') || 7 ;
         this._desiredTopLeftThickness =  splitViewThickness - dividerThickness - desiredThickness ;
       }
       
@@ -679,6 +694,11 @@ SC.SplitView = SC.View.extend(
   splitViewConstrainThickness: function(splitView, view, proposedThickness) {
     // console.log('%@.splitViewConstrainThickness(%@, %@, %@)'.fmt(this, splitView, view, proposedThickness));
     return proposedThickness;
-  }
+  },
+  
+  /* Force to rendering once the pane is attached */
+  _forceSplitCalculation: function(){
+    this.updateLayout(); 
+  }.observes('*pane.isPaneAttached')
 
 });

data/frameworks/sproutcore/frameworks/desktop/views/tab.js

@@ -111,14 +111,10 @@ SC.TabView = SC.View.extend(
       });
     }
     
-    view = this.containerView = this.createChildView(ContainerView, { 
-      rootElementPath: [0] 
-    }) ;
+    view = this.containerView = this.createChildView(ContainerView) ;
     childViews.push(view);
     
-    view = this.segmentedView = this.createChildView(this.segmentedView, { 
-      rootElementPath: [1] 
-    }) ;
+    view = this.segmentedView = this.createChildView(this.segmentedView) ;
     childViews.push(view);
     
     this.set('childViews', childViews);

data/frameworks/sproutcore/frameworks/foundation/controllers/array.js

@@ -284,7 +284,7 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     // if we can do this, then just forward the change.  This should fire
     // updates back up the stack, updating rangeObservers, etc.
     var content = this.get('content'); // note: use content, not observable
-    var objsToDestroy = [], i;
+    var objsToDestroy = [], i, objsLen;
     if (this.get('destroyOnRemoval')){
       for(i=0; i<amt; i++){
         objsToDestroy.push(content.objectAt(i+start));
@@ -292,7 +292,7 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     }
     
     if (content) content.replace(start, amt, objects);
-    for(i=0; i<objsToDestroy.length; i++){
+    for(i=0, objsLen = objsToDestroy.length; i<objsLen; i++){
       
       objsToDestroy[i].destroy();
     }
@@ -300,7 +300,12 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     
     return this; 
   },
-  
+
+  indexOf: function(object, startAt) {
+    var content = this._scac_observableContent();
+    return content ? content.indexOf(object, startAt) : -1;
+  },
+
   // ..........................................................
   // INTERNAL SUPPORT
   // 
@@ -366,11 +371,10 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     default:
       throw "%@.orderBy must be Array, String, or Function".fmt(this);
     }
-    
-    len = orderBy.get('length');
-    
+        
     // generate comparison function if needed - use orderBy
-    if (!func) {
+    if (!func) {  
+      len = orderBy.get('length');
       func = function(a,b) {
         var idx=0, status=0, key, aValue, bValue;
         for(idx=0;(idx<len)&&(status===0);idx++) {
@@ -456,11 +460,12 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     Whenever enumerable content changes, need to regenerate the 
     observableContent and notify that the range has changed.  
     
-    IMPORTANT: Assumes content is not null and is enumerable
+    This is called whenever the content enumerable changes or whenever orderBy
+    changes.
   */
   _scac_enumerableDidChange: function() {
     var content = this.get('content'), // use content directly
-        newlen  = content.get('length'),
+        newlen  = content ? content.get('length') : 0,
         oldlen  = this._scac_length;
         
     this._scac_length = newlen;
@@ -469,7 +474,7 @@ SC.ArrayController = SC.Controller.extend(SC.Array, SC.SelectionSupport,
     this.enumerableContentDidChange(0, newlen, newlen-oldlen);
     this.endPropertyChanges();
     this.updateSelectionAfterContentChange();
-  },
+  }.observes('orderBy'),
   
   /** @private
     Whenever array content changes, need to simply forward notification.

data/frameworks/sproutcore/frameworks/foundation/controllers/tree.js

@@ -99,7 +99,26 @@ SC.TreeController = SC.ObjectController.extend(SC.SelectionSupport,
   
   _sctc_arrangedObjectsContentDidChange: function() {
     this.updateSelectionAfterContentChange();
-  }.observes('*arrangedObjects.[]')
+  }.observes('*arrangedObjects.[]'),
+  
+  /**
+    @private
+    
+    Returns the first item in arrangeObjects that is not a group.  This uses
+    a brute force approach right now; we assume you probably don't have a lot
+    of groups up front.
+  */
+  firstSelectableObject: function() {
+    var objects = this.get('arrangedObjects'),
+        indexes, len, idx     = 0;
+        
+    if (!objects) return null; // fast track
+    
+    indexes = objects.contentGroupIndexes(null, objects);
+    len = objects.get('length');
+    while(indexes.contains(idx) && (idx<len)) idx++;
+    return idx>=len ? null : objects.objectAt(idx);
+  }.property()
   
 });
 

data/frameworks/sproutcore/frameworks/foundation/debug/control_test_pane.js

@@ -43,7 +43,7 @@ SC.ControlTestPane = SC.Pane.extend(
 /** @scope SC.ControlTestPane.prototype */ {
   
   classNames: ['sc-control-test-pane'],
-  layout: { right: 5, width: 350, top: 75, bottom: 5 },
+  layout: { right: 20, width: 350, top: 65, bottom: 5 },
 
   /**
     The starting top location for the first row.  This will increment as 

data/frameworks/sproutcore/frameworks/foundation/english.lproj/bootstrap.rhtml

@@ -6,15 +6,12 @@
 # License:   Licened under MIT license (see license.js)
 # ==========================================================================
 #
-# This file contains bootstrap code to be included at the top of standard 
-# SproutCore apps.  This code will autodetect the browser and then adds
-# a class name to the 'body' tag.  You can use this class name to customize
-# the CSS rules to be applied to your page based on browser.  The class name
-# contains the name webkit|mozilla|ie + the major version number
+# IMPORTANT: This bootstrap code is DEPRECATED.  Instead see the bootstrap 
+# framework in sproutcore for all new apps.
 #
 -%>
 <% content_for('bootstrap') do %>
-var SC = SC || { BUNDLE_INFO: {} };
+var SC = SC || { BUNDLE_INFO: {}, LAZY_INSTANTIATION: {} };
 SC.browser = (function() {
   var userAgent = navigator.userAgent.toLowerCase();
   var version = (userAgent.match( /.+(?:rv|it|ra|ie)[\/: ]([\d.]+)/ ) || [])[1] ;

data/frameworks/sproutcore/frameworks/foundation/english.lproj/button_view.css

@@ -8,6 +8,9 @@
   display: inline-block ;
   vertical-align: middle;
   text-decoration: none ;
+  border: 1px #555 solid;
+  border-radius: 3px;
+  -moz-border-radius: 3px;
 }
 
 .sc-button-view .sc-button-inner {

data/frameworks/sproutcore/frameworks/foundation/english.lproj/core.css

@@ -382,6 +382,10 @@ input[type="submit"] {
   top: 25px;
   background-color: #ccc;
   opacity: 0.4;
+  filter: progid:DXImageTransform.Microsoft.Alpha(Opacity=40);
+  -ms-filter: "progid:DXImageTransform.Microsoft.Alpha(Opacity=40)"; 
+  -moz-opacity:0.4;
+  
   position: relative;
   border-top: 1px solid #aaa;
   border-bottom: 1px solid #aaa;
@@ -431,6 +435,10 @@ input[type="submit"] {
 .sc-benchmark-row.even
 {
   opacity: 0.4;
+  filter: progid:DXImageTransform.Microsoft.Alpha(Opacity=40);
+  -ms-filter: "progid:DXImageTransform.Microsoft.Alpha(Opacity=40)"; 
+  -moz-opacity:0.4;
+  
   background-color: #ddd; 
 }
 

data/frameworks/sproutcore/frameworks/foundation/english.lproj/inline_editor.css

@@ -0,0 +1,12 @@
+.mozilla .sc-view .sc-text-field-view.inline-editor .border,
+.webkit .sc-view .sc-text-field-view.inline-editor .border,
+.msie .sc-view .sc-text-field-view.inline-editor .border,
+.sc-view .sc-text-field-view.inline-editor .border {
+  -webkit-appearance: none;
+  -moz-appearance: none;
+  border: none;
+}
+
+.mozilla .sc-view .sc-text-field-view.inline-editor.focus {
+  outline-offset: 0px;
+}

data/frameworks/sproutcore/frameworks/foundation/english.lproj/label.css

@@ -28,4 +28,15 @@
 .sc-label-view img.icon {
 	position: relative;
 	vertical-align: middle;
+}
+
+.sc-label-view .sc-hint {
+	position: absolute;
+	top: 3px;
+	left: 1px;
+	right: 1px;
+	bottom: 3px;
+	padding: 0px;
+	color: #aaa ;
+	font-size: 12px;
 }

data/frameworks/sproutcore/frameworks/foundation/english.lproj/text_field.css

@@ -12,6 +12,7 @@
   outline: auto 7px -webkit-focus-ring-color; 
   outline-offset: -2px;
 }
+
 .mozilla .sc-view .sc-text-field-view.focus {
   outline-color:-moz-mac-focusring;
   outline-offset:-5px;
@@ -86,6 +87,18 @@
   	top: 3px;
 }
 
+
+.sc-text-field-view input {
+	border: none;
+	outline: none;
+	}
+
+.sc-text-field-view textarea {
+	border: none;
+	outline: none;
+}
+
+
 .sc-text-field-view textarea {
   position: absolute;
 	top: 0px;

data/frameworks/sproutcore/frameworks/foundation/mixins/button.js

@@ -104,8 +104,7 @@ SC.Button = {
   
   /**
     The button icon.  Set this to either a URL or a CSS class name (for 
-    spriting).  To display an icon, you must set hasIcon to YES when the 
-    button is created.  Note that if you pass a URL, it must contain at 
+    spriting).  Note that if you pass a URL, it must contain at 
     least one slash to be detected as such.
     
     @property {String}