From 2c580e75a2e148e3bc06aa978b2ac93a6e03c007 2011-08-17 18:50:57
From: Brian E. Granger <ellisonbg@gmail.com>
Date: 2011-08-17 18:50:57
Subject: [PATCH] Don't scroll to bottom when last cell is selected.

---

diff --git a/IPython/frontend/html/notebook/static/js/cell.js b/IPython/frontend/html/notebook/static/js/cell.js
index df0e2ce..accc91e 100644
--- a/IPython/frontend/html/notebook/static/js/cell.js
+++ b/IPython/frontend/html/notebook/static/js/cell.js
@@ -24,10 +24,6 @@ var IPython = (function (IPython) {
     Cell.prototype.select = function () {
         this.element.addClass('ui-widget-content ui-corner-all');
         this.selected = true;
-        // TODO: we need to test across browsers to see if both of these are needed.
-        // In the meantime, there should not be any harm in having them both.
-        this.element.find('textarea').trigger('focusin');
-        this.element.find('textarea').trigger('focus');
     };
 
 
diff --git a/IPython/frontend/html/notebook/static/js/notebook.js b/IPython/frontend/html/notebook/static/js/notebook.js
index 64b2eb4..b7290f4 100644
--- a/IPython/frontend/html/notebook/static/js/notebook.js
+++ b/IPython/frontend/html/notebook/static/js/notebook.js
@@ -31,7 +31,8 @@ var IPython = (function (IPython) {
         // i) provide a margin between the last cell and the end of the notebook
         // ii) to prevent the div from scrolling up when the last cell is being
         // edited, but is too low on the page, which browsers will do automatically.
-        this.element.append($('<div class="end_space"></div>').height(150));
+        var end_space = $('<div class="end_space"></div>').height(150);
+        this.element.append(end_space);
         $('div#notebook').addClass('border-box-sizing');
     };
 
@@ -155,9 +156,6 @@ var IPython = (function (IPython) {
                 this.selected_cell().unselect();
             };
             this.cells()[index].select();
-            if (index === (this.ncells()-1)) {
-                this.scroll_to_bottom();
-            };
         };
         return this;
     };