sproutcore 1.8.1 → 1.8.2.1

data/VERSION.yml

@@ -1,4 +1,4 @@
 ---
 :major: 1
 :minor: 8
-:patch: 1
+:patch: 2.1

data/lib/frameworks/sproutcore/CHANGELOG.md

@@ -1,5 +1,5 @@
-CHANGE LOG FOR 1.6+
-===================
+CHANGE LOG
+==========
 
 Edge
 ----------
@@ -18,6 +18,21 @@ Edge
 
 ### BUG FIXES
 
+
+1.8.2 - BUG FIXES
+----------
+
+* Fixed syntax error in Datastore unit test. [c62a0a9]
+* SC.SplitView can now mixin SC.SplitChild. [6f1586c]
+* Thinned picker pane border divs so that they don't overlap the content view. [0b06db2]
+* Prevents target property conflict when configuring button targets with SC.AlertPane. [0364b5c]
+* Changed the aria-orientation of horizontal SC.ScrollView to 'horizontal' from 'vertical'. [c3ee69b]
+* Allows SC.CollectionFastPath to work with sparse content by always returning an item view even when content isn't yet available. [020653b]
+* Prevents SC.GridView from iterating over its content array in order to work with sparse content. [020653b]
+* The 'mobile-safari' body class name is no longer being added in all browsers. [b491224]
+* Enables pasting in SC.TextFieldView to notify that the value changed. [a51318b]
+* Prevents default touch behavior being intercepted on <textarea> and <select> elements. [8093963]
+
 1.8.1 - BUG FIXES
 ----------
 

data/lib/frameworks/sproutcore/frameworks/bootstrap/system/loader.js

@@ -13,7 +13,7 @@ SC.setupBodyClassNames = function() {
   if (!el) return ;
 
   browser = SC.browser.current ;
-  platform = SC.browser.windows ? 'windows' : SC.browser.mac ? 'mac' : 'other-platform' ;
+  platform = SC.browser.isWindows ? 'windows' : SC.browser.isMac ? 'mac' : 'other-platform' ;
   style = document.documentElement.style;
   shadows = (style.MozBoxShadow !== undefined) ||
                 (style.webkitBoxShadow !== undefined) ||
@@ -46,7 +46,7 @@ SC.setupBodyClassNames = function() {
   }
 
   if(browser==="safari" || browser==="chrome") classNames.push('webkit');
-  if (SC.browser.mobileSafari) classNames.push('mobile-safari') ;
+  if (SC.browser.isMobileSafari) classNames.push('mobile-safari') ;
   if ('createTouch' in document) classNames.push('touch');
   el.className = classNames.join(' ') ;
 } ;

data/lib/frameworks/sproutcore/frameworks/core_foundation/system/root_responder.js

@@ -1488,7 +1488,7 @@ SC.RootResponder = SC.Object.extend(
   ignoreTouchHandle: function(evt) {
     if(SC.browser.isMobileSafari){
       var tag = evt.target.tagName;
-      if(tag==="INPUT" || tag==="A"){
+      if(tag==="INPUT" || tag==="TEXTAREA" || tag==="A" || tag==="SELECT"){
         evt.allowDefault();
         return YES;
       }

data/lib/frameworks/sproutcore/frameworks/core_foundation/tests/system/root_responder/root_responder.js

@@ -82,6 +82,29 @@ test("root_responder.makeKeyPane() : Should make the main pane as the key pane i
  equals(responder.get('keyPane'),lightPane);
 });
 
+test("root_responder.ignoreTouchHandle() : Should ignore TEXTAREA, INPUT, A, and SELECT elements", function () {
+ var wasMobileSafari = SC.browser.isMobileSafari;
+ SC.browser.isMobileSafari = YES;
+
+ ["A", "INPUT", "TEXTAREA", "SELECT"].forEach(function (tag) {
+   ok(responder.ignoreTouchHandle({
+     target: { tagName: tag },
+     allowDefault: SC.K
+   }), "should pass touch events through to <" + tag + ">s");
+ });
+
+ ["AUDIO", "B", "Q", "BR", "BODY", "BUTTON", "CANVAS", "FORM",
+  "IFRAME", "IMG", "OPTION", "P", "PROGRESS", "STRONG",
+  "TABLE", "TBODY", "TR", "TH", "TD", "VIDEO"].forEach(function (tag) {
+   ok(!responder.ignoreTouchHandle({
+     target: { tagName: tag },
+     allowDefault: SC.K
+   }), "should NOT pass touch events through to <" + tag + ">s");
+ });
+
+ SC.browser.isMobileSafari = wasMobileSafari;
+});
+
 //// CLEANUP
 /// Commenting out this two functions as the methods don't exist
 //// confirm with Charles 

data/lib/frameworks/sproutcore/frameworks/datastore/tests/system/query/compare.js

@@ -4,22 +4,22 @@
 // License:   Licensed under MIT license (see license.js)
 // ==========================================================================
 /*globals module ok equals same test MyApp */
- 
+
 // test parsing of query string
 var store, storeKey, rec1, rec2, rec3, rec4, rec5, MyApp, q;
 module("SC.Query comparison of records", {
   setup: function() {
-    
+
     SC.RunLoop.begin();
-    
+
     // setup dummy app and store
     MyApp = SC.Object.create({
       store: SC.Store.create()
     });
-    
+
     // setup a dummy model
     MyApp.Foo = SC.Record.extend({});
-    
+
     // load some data
     MyApp.store.loadRecords(MyApp.Foo, [
       { guid: 1, firstName: "John", lastName: "Doe", year: 1974 },
@@ -28,9 +28,9 @@ module("SC.Query comparison of records", {
       { guid: 4, firstName: "Johnny", lastName: "Cash", active: false },
       { guid: 5, firstName: "Bert", lastName: "Berthold", active: true }
     ]);
-    
+
     SC.RunLoop.end();
-    
+
     rec1 = MyApp.store.find(MyApp.Foo,1);
     rec2 = MyApp.store.find(MyApp.Foo,2);
     rec3 = MyApp.store.find(MyApp.Foo,3);
@@ -51,42 +51,42 @@ module("SC.Query comparison of records", {
     rec18 = MyApp.store.createRecord(MyApp.Foo, { firstName: "Amie", lastName: "Frontenac", active: true });
     rec19 = MyApp.store.createRecord(MyApp.Foo, { firstName: "Amelie", lastName: "Auguste", active: true });
     rec20 = MyApp.store.createRecord(MyApp.Foo, { firstName: "Percy", lastName: "Douglas", active: true });
-    
+
     q = SC.Query.create();
   }
 });
- 
- 
+
+
 // ..........................................................
 // TESTS
-// 
+//
 
 test("parse() should work with conditions = null", function(){
   q.parse();
 });
- 
+
 test("building the order", function() {
   // undefined orderBy
   q.orderBy = null;
   q.parse();
   equals(q._order.length, 0, 'order should be empty');
-  
+
   // empty orderBy
   q.orderBy = "";
   q.parse();
   equals(q._order.length, 0, 'order should be empty');
-  
+
   // single property
   q.orderBy = "firstName";
   q.parse();
   equals(q._order[0].propertyName,'firstName', 'propertyName should be firstName');
-  
+
   // more properties
   q.orderBy = "lastName, firstName";
   q.parse();
   equals(q._order[0].propertyName,'lastName', 'propertyName should be lastName');
   equals(q._order[1].propertyName,'firstName', 'propertyName should be firstName');
-  
+
   // more properties with direction
   q.orderBy = "lastName, firstName, year DESC";
   q.parse();
@@ -102,6 +102,8 @@ test("no order should result in comparison by guid", function() {
   q.orderBy = null;
   q.parse();
   equals(q.compare(rec1,rec2), -1, 'guid 1 should be before guid 2');
+});
+
 /**
   This test was added to prove new code that switched from ordering by guid
   if there is no orderBy property on the query to ordering by storeKey if
@@ -147,34 +149,34 @@ test("comparing non existent properties", function() {
   q.parse();
   equals(q.compare(rec5,rec1), -1, 'null should be before 1974');
 });
- 
+
 test("comparing null and boolean properties", function() {
   q.orderBy = "active";
   q.parse();
   equals(q.compare(rec3,rec4), -1, 'null should be before false');
   equals(q.compare(rec4,rec5), -1, 'false should be before true');
 });
- 
+
 test("comparing number properties", function() {
   q.orderBy = "year";
   q.parse();
   equals(q.compare(rec1,rec2), -1, '1974 should be before 1975');
-  
+
   q.orderBy = "year DESC";
   q.parse();
   equals(q.compare(rec1,rec2), 1, '1974 should be after 1975 with DESC');
-}); 
- 
- 
+});
+
+
 test("comparing string properties", function() {
   q.orderBy = "firstName";
   q.parse();
   equals(q.compare(rec1,rec2), 1, 'John should be after Jane');
-  
+
   q.orderBy = "firstName DESC";
   q.parse();
   equals(q.compare(rec1,rec2), -1, 'John should be before Jane with DESC');
-}); 
+});
 
 test("comparing by equal properties should use guid for order", function() {
   q.orderBy = "lastName";

data/lib/frameworks/sproutcore/frameworks/desktop/mixins/collection_fast_path.js

@@ -5,9 +5,9 @@
 // License:   Licensed under MIT license (see license.js)
 // ==========================================================================
 
-/** 
+/**
   @namespace
-  
+
   An experimental `CollectionView` mixin that makes it extremely fast under
   certain circumstances, including for mobile devices.
 */
@@ -19,12 +19,12 @@ SC.CollectionFastPath = {
   initMixin: function() {
     this._indexMap = {};
   },
-  
+
   /**
     Returns the pool for a given example view.
-    
+
     The pool is calculated based on the guid for the example view class.
-    
+
     @param {SC.View} exampleView
   */
   poolForExampleView: function(exampleView) {
@@ -32,31 +32,31 @@ SC.CollectionFastPath = {
     if (!this[poolKey]) this[poolKey] = [];
     return this[poolKey];
   },
-  
+
   /**
     Creates an item view from a given example view, configuring it with basic settings
     and the supplied attributes.
-    
+
     @param {SC.View} exampleView
     @param {Hash} attrs
   */
   createItemViewFromExampleView: function(exampleView, attrs) {
     // create the example view
     var ret = this.createItemView(exampleView, null, attrs);
-    
+
     // for our pooling, if it is poolable, mark the view as poolable and
     // give it a reference to its pool.
     if (ret.isPoolable) {
       ret.owningPool = this.poolForExampleView(exampleView);
     }
-    
+
     // we will sometimes need to know what example view created the item view
     ret.createdFromExampleView = exampleView;
 
     // and now, return (duh)
     return ret;
   },
-  
+
   /**
     @param {SC.View} itemView
     @param {Hash} attrs
@@ -78,7 +78,7 @@ SC.CollectionFastPath = {
     itemView.setIfChanged('page', this.page);
     itemView.endPropertyChanges();
   },
-  
+
   /**
     Configures a pooled view, calling `.awakeFromPool` if it is defined.
 
@@ -88,15 +88,15 @@ SC.CollectionFastPath = {
   wakePooledView: function(itemView, attrs) {
     // configure
     this.configureItemView(itemView, attrs);
-    
+
     // awake from the pool, etc.
     if (itemView.awakeFromPool) itemView.awakeFromPool(itemView.owningPool, this);
   },
-  
+
   /**
     Gets an item view from an example view, from a pool if possible, and otherwise
     by generating it.
-    
+
     @param {SC.View} exampleView
     @param {Hash} attrs
   */
@@ -104,7 +104,7 @@ SC.CollectionFastPath = {
     // we will try to get it from a pool. This will fill ret. If ret is not
     // filled, then we'll know to generate one.
     var ret;
-    
+
     // if it is poolable, we just grab from the pool.
     if (exampleView.prototype.isPoolable) {
       var pool = this.poolForExampleView(exampleView);
@@ -113,18 +113,18 @@ SC.CollectionFastPath = {
         this.wakePooledView(ret, attrs);
       }
     }
-    
+
     if (!ret) {
       ret = this.createItemViewFromExampleView(exampleView, attrs);
     }
-    
+
     return ret;
   },
-  
+
   /**
     Releases an item view. If the item view is pooled, it puts it into the pool;
     otherwise, this calls `.destroy()`.
-    
+
     This is called for one of two purposes: to release a view that is no longer displaying,
     or to release an older cached version of a view that needed to be replaced because the
     example view changed.
@@ -137,30 +137,30 @@ SC.CollectionFastPath = {
       itemView.destroy();
       return;
     }
-    
+
     // otherwise, we need to return to view
     var pool = itemView.owningPool;
     pool.push(itemView);
     if (itemView.hibernateInPool) itemView.hibernateInPool(pool, this);
   },
-  
+
   /** @private
     Returns YES if the item at the index is a group.
   */
   contentIndexIsGroup: function(view, content, index) {
     var contentDelegate = this.get("contentDelegate");
-    
+
     // setup our properties
     var groupIndexes = this.get('_contentGroupIndexes'), isGroupView = NO;
-    
+
     // and do our checking
     isGroupView = groupIndexes && groupIndexes.contains(index);
     if (isGroupView) isGroupView = contentDelegate.contentIndexIsGroup(this, this.get("content"), index);
-    
+
     // and return
     return isGroupView;
   },
-  
+
   /** @private
     Determines the example view for a content index. There are two optional parameters that will
     speed things up: `contentObject` and `isGroupView`. If you don't supply them, they must be computed.
@@ -182,24 +182,24 @@ SC.CollectionFastPath = {
       if (key && item) ExampleView = item.get(key);
       if (!ExampleView) ExampleView = this.get('exampleView');
     }
-    
+
     return ExampleView;
   },
-  
+
   /** @private
     This may seem somewhat awkward, but it is for memory performance: this fills in a hash
     YOU provide with the properties for the given content index.
-    
+
     Properties include both the attributes given to the view and some `CollectionView` tracking
     properties, most importantly the exampleView.
   */
   setAttributesForItem: function(item, index, attrs) {
-    var del = this.get('contentDelegate'), 
+    var del = this.get('contentDelegate'),
         isGroupView = this.contentIndexIsGroup(this, this.get('content'), index),
         ExampleView = this.exampleViewForItem(item, index),
         content = this.get("content");
-    
-    // 
+
+    //
     // FIGURE OUT "NORMAL" ATTRIBUTES
     //
     attrs.createdFromExampleView = ExampleView;
@@ -218,11 +218,11 @@ SC.CollectionFastPath = {
     attrs.layout = this.layoutForContentIndex(index);
     if (!attrs.layout) attrs.layout = ExampleView.prototype.layout;
   },
-  
+
   //
   // ITEM LOADING/DOM MANAGEMENT
   //
-  
+
   /** @private
     Returns mapped item views for the supplied item.
   */
@@ -230,9 +230,9 @@ SC.CollectionFastPath = {
     if (!map) map = this._viewMap;
     return map[SC.guidFor(item)];
   },
-  
+
   /** @private
-    Returns the mapped view for an item at the specified index. 
+    Returns the mapped view for an item at the specified index.
   */
   mappedViewForItem: function(item, idx, map) {
     if (!map) map = this._viewMap;
@@ -240,24 +240,24 @@ SC.CollectionFastPath = {
     if (!m) return undefined;
     return m[idx];
   },
-  
+
   /** @private
     Maps a view to an item/index combination.
   */
   mapView: function(item, index, view, map) {
     // get the default view map if a map was not supplied
     if (!map) map = this._viewMap;
-    
+
     // get the item map
     var g = SC.guidFor(item),
         imap = map[g];
     if (!imap) imap = map[g] = {_length: 0};
-    
+
     // fill in the index
     imap[index] = view;
     imap._length++;
   },
-  
+
   /** @private
     Unmaps a view from an item/index combination.
   */
@@ -265,20 +265,20 @@ SC.CollectionFastPath = {
     if (!map) map = this._viewMap;
     var g = SC.guidFor(item),
         imap = map[g];
-    
+
     // return if there is nothing to do
     if (!imap) return;
-    
+
     // remove
     if (imap[index]) {
       var v = imap[index];
       delete imap[index];
-      
+
       imap._length--;
       if (imap._length <= 0) delete map[g];
     }
   },
-  
+
   /**
     Returns the item view for the given content index.
     NOTE: THIS WILL ADD THE VIEW TO DOM TEMPORARILY (it will be cleaned if
@@ -289,53 +289,52 @@ SC.CollectionFastPath = {
   itemViewForContentIndex: function(index) {
     var content = this.get("content");
     if (!content) return;
-    
+
     var item = content.objectAt(index);
-    if (!item) return null;
-    
+
     var exampleView = this.exampleViewForItem(item, index),
         view = this._indexMap[index];
-    
+
     if (view && view.createdFromExampleView !== exampleView) {
       this.removeItemView(view);
       this.unmapView(item, index);
       view = null;
     }
-    
+
     if (!view) {
       view = this.addItemView(exampleView, item, index);
     }
-    
+
     return view;
   },
-  
+
   /** @private
     Returns the nearest item view index to the supplied index mapped to the item.
   */
   nearestMappedViewIndexForItem: function(item, index, map) {
     var m = this.mappedViewsForItem(item, map);
     if (!m) return null;
-    
+
     // keep track of nearest and the nearest distance
     var nearest = null, ndist = -1, dist = 0;
-    
+
     // loop through
     for (var idx in m) {
       idx = parseInt(idx, 10);
       if (isNaN(idx)) continue;
       // get distance
       dist = Math.abs(index - idx);
-      
+
       // compare to nearest distance
       if (ndist < 0 || dist < ndist) {
         ndist = dist;
         nearest = idx;
       }
     }
-    
+
     return nearest;
   },
-  
+
   /** @private
     Remaps the now showing views to their new indexes (if they have moved).
   */
@@ -353,11 +352,11 @@ SC.CollectionFastPath = {
     // first, find items which we can (that already exist, etc.)
     nowShowing.forEach(function(idx) {
       item = content.objectAt(idx);
-      
+
       // determine if we have view(s) in the old map for the item
       var possibleExistingViews = this.mappedViewsForItem(item, oldMap);
       if (possibleExistingViews) {
-        
+
         // if it is the same index, we just take it. End of story.
         if (possibleExistingViews[idx]) {
           var v = possibleExistingViews[idx];
@@ -373,14 +372,14 @@ SC.CollectionFastPath = {
         itemsToAdd.push(idx);
       }
     }, this);
-    
+
     // now there are also some items which _could_ exist (but might not!)
     for (var idx = 0, len = mayExist.length; idx < len; idx++) {
       var newIdx = mayExist[idx];
       item = content.objectAt(newIdx);
       var nearestOldIndex = this.nearestMappedViewIndexForItem(item, newIdx, oldMap),
           nearestView;
-      
+
       if (!SC.none(nearestOldIndex)) {
         nearestView = this.mappedViewForItem(item, nearestOldIndex, oldMap);
         var newExampleView = this.exampleViewForItem(item, newIdx);
@@ -397,10 +396,10 @@ SC.CollectionFastPath = {
         itemsToAdd.push(newIdx);
       }
     }
-    
+
     return oldMap;
   },
-  
+
   /**
     Reloads.
 
@@ -409,57 +408,57 @@ SC.CollectionFastPath = {
   */
   reloadIfNeeded: function(nowShowing, scrollOnly) {
     var content = this.get("content"), invalid;
-    
+
     // we use the nowShowing to determine what should and should not be showing.
     if (!nowShowing || !nowShowing.isIndexSet) nowShowing = this.get('nowShowing');
-    
+
     // we only update if this is a non-scrolling update.
     // don't worry: we'll actually update after the fact, and the invalid indexes should
     // be queued up nicely.
     if (!scrollOnly) {
       invalid = this._invalidIndexes;
       if (!invalid || !this.get('isVisibleInWindow')) return this;
-      this._invalidIndexes = NO; 
-      
+      this._invalidIndexes = NO;
+
       // tell others we will be reloading
       if (invalid.isIndexSet && invalid.contains(nowShowing)) invalid = YES ;
       if (this.willReload) this.willReload(invalid === YES ? null : invalid);
     }
-    
+
     // get arrays of items to add/remove
     var itemsToAdd = this._itemsToAdd || (this._itemsToAdd = []);
-    
+
     // remap
     var oldMap = this.remapItemViews(nowShowing);
-    
+
     // The oldMap has the items to remove, so supply it to processRemovals
     this.processRemovals(oldMap);
-    
+
     // handle the invalid set (if it is present)
     if (invalid) {
       this.processUpdates(invalid === YES ? nowShowing : invalid);
     }
-    
+
     // process items to add
     this.processAdds();
-    
+
     // only clear the DOM pools if this is not during scrolling. Adding/removing is a
     // bad idea while scrolling :)
     if (!scrollOnly) this.clearDOMPools();
-    
+
     // clear the lists
     itemsToAdd.length = 0;
-    
+
     // and if this is a full reload, we need to adjust layout
     if (!scrollOnly) {
       var layout = this.computeLayout();
       if (layout) this.adjust(layout);
       if (this.didReload) this.didReload(invalid === YES ? null : invalid);
     }
-    
+
     return this;
   },
-  
+
   /**
     Loops through remove queue and removes.
 
@@ -472,17 +471,17 @@ SC.CollectionFastPath = {
       for (var itemIdx in imap) {
         itemIdx = parseInt(itemIdx, 10);
         if (isNaN(itemIdx)) continue;
-        
+
         var view = imap[itemIdx];
-        
+
         if (this._indexMap[itemIdx] === view) delete this._indexMap[itemIdx];
-        
+
         view._isInCollection = NO;
         this.removeItemView(view);
       }
     }
   },
-  
+
   /** @private
     Loops through update queue and... updates.
   */
@@ -497,23 +496,23 @@ SC.CollectionFastPath = {
       }
     }, this);
   },
-  
+
   /** @private
     Loops through add queue and, well, adds.
   */
   processAdds: function() {
     var content = this.get("content");
-    
+
     var add = this._itemsToAdd, idx, len = add.length, itemIdx, item;
     for (idx = 0; idx < len; idx++) {
       itemIdx = add[idx]; item = content.objectAt(itemIdx);
-      
+
       // get example view and create item view
       var exampleView = this.exampleViewForItem(item, itemIdx);
       var view = this.addItemView(exampleView, item, itemIdx);
     }
   },
-  
+
   /** @private
     Clear all DOM pools.
   */
@@ -523,34 +522,34 @@ SC.CollectionFastPath = {
       this.clearDOMPool(pools[p]);
     }
   },
-  
+
   /**
     @type Number
     @default 10
   */
   domPoolSize: 10,
-  
+
   /** @private
     Clears a specific DOM pool.
   */
   clearDOMPool: function(pool) {
     var idx, len = pool.length, item;
-    
+
     // we skip one because there is a buffer area of one while scrolling
     for (idx = this.domPoolSize; idx < len; idx++) {
       item = pool[idx];
 
       // remove from DOM
       this.removeChild(item);
-      
+
       // release the item
       this.releaseItemView(item);
     }
-    
+
     // pool is cleared.
     pool.length = Math.min(pool.length, this.domPoolSize);
   },
-  
+
   /** @private
     Returns the DOM pool for the given exampleView.
   */
@@ -560,7 +559,7 @@ SC.CollectionFastPath = {
     if (!pool) pool = pools[guid] = [];
     return pool;
   },
-  
+
   /** @private
     Tries to find an item for the given example view in a dom pool.
     If one could not be found, returns `null`.
@@ -572,7 +571,7 @@ SC.CollectionFastPath = {
     if (view.wakeFromDOMPool) view.wakeFromDOMPool();
     return view;
   },
-  
+
   /** @private
     Sends a view to a DOM pool.
   */
@@ -584,48 +583,48 @@ SC.CollectionFastPath = {
     view.set("layerId", SC.guidFor(view));
     if (view.sleepInDOMPool) view.sleepInDOMPool();
   },
-  
+
   /** @private
     Adds an item view (grabbing the actual item from one of the pools if possible).
   */
   addItemView: function(exampleView, object, index) {
     var view, attrs = this._TMP_ATTRS || (this._TMP_ATTRS = {});
-    
+
     // in any case, we need attributes
     this.setAttributesForItem(object, index, attrs);
-    
+
     // try to get from DOM pool first
     if (view = this.itemFromDOMPool(exampleView)) {
       // set attributes
       this.configureItemView(view, attrs);
-      
+
       // set that it is in the collection
       view._isInCollection = YES;
-      
+
       // add to view map (if not used, it will be removed)
       this.mapView(object, index, view);
       this._indexMap[index] = view;
-      
+
       // and that should have repositioned too
       return view;
     }
-    
+
     // otherwise, just allocate a view
     view = this.allocateItemView(exampleView, attrs);
-    
+
     // and then, add it
     this.appendChild(view);
-    
+
     // set that it is in the collection.
     view._isInCollection = YES;
-    
+
     // add to view map (if not used, it will be removed)
     this.mapView(object, index, view);
     this._indexMap[index] = view;
-    
+
     return view;
   },
-  
+
   /** @private
     Removes an item view.
   */
@@ -637,11 +636,11 @@ SC.CollectionFastPath = {
     }
     current._isInCollection = NO;
   },
-  
+
   /**
     Updates the specified item view. If the view is not "layer cacheable" or the
     example view has changed, it will be redrawn.
-    
+
     Otherwise, nothing will happen.
   */
   updateItemView: function(current, exampleView, object, index) {
@@ -650,7 +649,7 @@ SC.CollectionFastPath = {
       this.unmapView(current, index);
       delete this._indexMap[index];
       this.removeItemView(current, object, index);
-      
+
       // add new and map
       var newView = this.addItemView(exampleView, object, index);
     } else {
@@ -660,8 +659,8 @@ SC.CollectionFastPath = {
       this.configureItemView(current, attrs);
     }
   },
-  
-  
+
+
   /** @private
     Tells `ScrollView` that this should receive live updates during touch scrolling.
     We are so fast, aren't we?
@@ -673,39 +672,39 @@ SC.CollectionFastPath = {
 
   /** @private */
   _tolerance: 100,
-  
+
   /** @private */
   touchScrollDidChange: function(left, top) {
     // prevent getting too many in close succession.
     if (Date.now() - this._lastTouchScrollTime < 25) return;
-    
+
     var clippingFrame = this.get('clippingFrame');
-    
+
     var cf = this._inScrollClippingFrame || (this._inScrollClippingFrame = {x: 0, y: 0, width: 0, height: 0});
     cf.x = clippingFrame.x; cf.y = clippingFrame.y; cf.width = clippingFrame.width; cf.height = clippingFrame.height;
-    
+
     // update
     cf.x = left;
     cf.y = top;
-    
+
     var r = this.contentIndexesInRect(cf);
     if (!r) return; // no rect, do nothing.
-    
-    var len = this.get('length'), 
+
+    var len = this.get('length'),
         max = r.get('max'), min = r.get('min');
 
     if (max > len || min < 0) {
       r = r.copy();
       r.remove(len, max-len).remove(min, 0-min).freeze();
     }
-    
+
     if (this._lastNowShowing) {
       if (r.contains(this._lastNowShowing) && this._lastNowShowing.contains(r)) return;
     }
     this._lastNowShowing = r;
     this.reloadIfNeeded(r, YES);
-    
+
     this._lastTouchScrollTime = Date.now();
   }
-  
+
 };