From 21bbe1d11a2ff27f0e5a5f5838ea4e87947a3149 Mon Sep 17 00:00:00 2001
From: Vivian Fang <vivian98.fang@gmail.com>
Date: Wed, 6 Apr 2016 17:14:46 -0700
Subject: [PATCH 1/3] adding an action for toggling all line numbers, with
 keyboard shortcut Shift-L

---
 notebook/static/notebook/js/actions.js         |  8 ++++++++
 notebook/static/notebook/js/keyboardmanager.js |  1 +
 notebook/static/notebook/js/maintoolbar.js     |  1 +
 notebook/static/notebook/js/notebook.js        | 12 ++++++++++++
 4 files changed, 22 insertions(+)

diff --git a/notebook/static/notebook/js/actions.js b/notebook/static/notebook/js/actions.js
index 83df33bee..0c479f5bb 100644
--- a/notebook/static/notebook/js/actions.js
+++ b/notebook/static/notebook/js/actions.js
@@ -456,6 +456,14 @@ define(function(require){
                 env.notebook.show_command_palette();
             }
         },
+        'toggle-all-line-numbers': {
+            help : 'toggles line numbers in all cells',
+            icon: 'fa-list-ol',
+            handler: function(env) {
+                console.log('calling function');
+                env.notebook.toggle_all_line_numbers();
+            }
+        },
         'toggle-toolbar':{
             help: 'hide/show the toolbar',
             handler : function(env){
diff --git a/notebook/static/notebook/js/keyboardmanager.js b/notebook/static/notebook/js/keyboardmanager.js
index 4edb29935..f2598db23 100644
--- a/notebook/static/notebook/js/keyboardmanager.js
+++ b/notebook/static/notebook/js/keyboardmanager.js
@@ -159,6 +159,7 @@ define([
             'o' : 'jupyter-notebook:toggle-cell-output-collapsed',
             's' : 'jupyter-notebook:save-notebook',
             'l' : 'jupyter-notebook:toggle-cell-line-numbers',
+            'shift-l' : 'jupyter-notebook:toggle-all-line-numbers',
             'h' : 'jupyter-notebook:show-keyboard-shortcuts',
             'z' : 'jupyter-notebook:undo-cell-deletion',
             'q' : 'jupyter-notebook:close-pager',
diff --git a/notebook/static/notebook/js/maintoolbar.js b/notebook/static/notebook/js/maintoolbar.js
index f449afa17..4ee0b2ce2 100644
--- a/notebook/static/notebook/js/maintoolbar.js
+++ b/notebook/static/notebook/js/maintoolbar.js
@@ -55,6 +55,7 @@ define([
             'run_int'],
          ['<add_celltype_list>'],
          [['jupyter-notebook:show-command-palette']],
+         [['jupyter-notebook:toggle-all-line-numbers']],
          ['<add_celltoolbar_reminder>']
         ];
         this.construct(grps);
diff --git a/notebook/static/notebook/js/notebook.js b/notebook/static/notebook/js/notebook.js
index 250db8bf8..843566b52 100644
--- a/notebook/static/notebook/js/notebook.js
+++ b/notebook/static/notebook/js/notebook.js
@@ -65,6 +65,7 @@ define(function (require) {
         this.ws_url = options.ws_url;
         this._session_starting = false;
         this.last_modified = null;
+        this.line_numbers = false;
         // debug 484
         this._last_modified = 'init';
         // Firefox workaround
@@ -552,6 +553,17 @@ define(function (require) {
         }
         return result;
     };
+    
+    /**
+     * Toggles the display of line numbers in all cells.
+     */
+    Notebook.prototype.toggle_all_line_numbers = function () {
+        this.line_numbers = !this.line_numbers;
+        var display = this.line_numbers;
+        this.get_cells().map(function(c) {
+            c.code_mirror.setOption('lineNumbers', display);
+        })
+    }
 
     /**
      * Get the cell above a given cell.

From ac526c2b552062bfc4dd9f3dcd36f8f13dbf4597 Mon Sep 17 00:00:00 2001
From: Grant Nestor <grantnestor@gmail.com>
Date: Wed, 10 Aug 2016 18:50:38 -0700
Subject: [PATCH 2/3] Move toggle line numbers from toolbar to "View" menu

---
 notebook/static/notebook/js/maintoolbar.js |  1 -
 notebook/static/notebook/js/menubar.js     |  1 +
 notebook/templates/notebook.html           | 10 ++++++++--
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/notebook/static/notebook/js/maintoolbar.js b/notebook/static/notebook/js/maintoolbar.js
index 4ee0b2ce2..f449afa17 100644
--- a/notebook/static/notebook/js/maintoolbar.js
+++ b/notebook/static/notebook/js/maintoolbar.js
@@ -55,7 +55,6 @@ define([
             'run_int'],
          ['<add_celltype_list>'],
          [['jupyter-notebook:show-command-palette']],
-         [['jupyter-notebook:toggle-all-line-numbers']],
          ['<add_celltoolbar_reminder>']
         ];
         this.construct(grps);
diff --git a/notebook/static/notebook/js/menubar.js b/notebook/static/notebook/js/menubar.js
index 04984803a..5b032d06a 100644
--- a/notebook/static/notebook/js/menubar.js
+++ b/notebook/static/notebook/js/menubar.js
@@ -211,6 +211,7 @@ define([
             '#move_cell_down': 'move-cell-down',
             '#toggle_header': 'toggle-header',
             '#toggle_toolbar': 'toggle-toolbar',
+            '#toggle_line_numbers': 'toggle-all-line-numbers',
             '#insert_cell_above': 'insert-cell-above',
             '#insert_cell_below': 'insert-cell-below',
             '#run_cell': 'run-cell',
diff --git a/notebook/templates/notebook.html b/notebook/templates/notebook.html
index f8e4f3c07..52cb74cec 100644
--- a/notebook/templates/notebook.html
+++ b/notebook/templates/notebook.html
@@ -154,10 +154,16 @@ data-notebook-path="{{notebook_path | urlencode}}"
                     <ul id="view_menu" class="dropdown-menu">
                         <li id="toggle_header"
                             title="Show/Hide the logo and notebook title (above menu bar)">
-                            <a href="#">Toggle Header</a></li>
+                            <a href="#">Toggle Header</a>
+                        </li>
                         <li id="toggle_toolbar"
                             title="Show/Hide the action icons (below menu bar)">
-                            <a href="#">Toggle Toolbar</a></li>
+                            <a href="#">Toggle Toolbar</a>
+                        </li>
+                        <li id="toggle_line_numbers"
+                            title="Show/Hide line numbers in cells">
+                            <a href="#">Toggle Line Numbers</a>
+                        </li>
                         <li id="menu-cell-toolbar" class="dropdown-submenu">
                             <a href="#">Cell Toolbar</a>
                             <ul class="dropdown-menu" id="menu-cell-toolbar-submenu"></ul>

From 14089ce13941cc110a37993a700b1e1d242baf3d Mon Sep 17 00:00:00 2001
From: Matthias Bussonnier <bussonniermatthias@gmail.com>
Date: Tue, 3 May 2016 14:44:29 -0700
Subject: [PATCH 3/3] Inherit configuration and propagate it from MD cell

Persist the configuration of toggling in nbconfig.
---
 notebook/static/notebook/js/actions.js  | 17 ++++++++++++++---
 notebook/static/notebook/js/cell.js     | 12 +++++++++---
 notebook/static/notebook/js/codecell.js |  2 +-
 notebook/static/notebook/js/notebook.js | 22 +++++++++++++++++-----
 notebook/static/notebook/js/textcell.js |  6 +++---
 5 files changed, 44 insertions(+), 15 deletions(-)

diff --git a/notebook/static/notebook/js/actions.js b/notebook/static/notebook/js/actions.js
index 6465f959b..d61e9ee75 100644
--- a/notebook/static/notebook/js/actions.js
+++ b/notebook/static/notebook/js/actions.js
@@ -462,12 +462,23 @@ define(function(require){
                 env.notebook.show_command_palette();
             }
         },
+        'show-all-line-numbers': {
+            help : 'show line numbers in all cells, and persist the setting',
+            handler: function(env) {
+                env.notebook.line_numbers = true;
+            }
+        },
+        'hide-all-line-numbers': {
+            help : 'hide line numbers in all cells, and persist the setting',
+            handler: function(env) {
+                env.notebook.line_numbers = false;
+            }
+        },
         'toggle-all-line-numbers': {
-            help : 'toggles line numbers in all cells',
+            help : 'toggles line numbers in all cells, and persist the setting',
             icon: 'fa-list-ol',
             handler: function(env) {
-                console.log('calling function');
-                env.notebook.toggle_all_line_numbers();
+                env.notebook.line_numbers = !env.notebook.line_numbers;
             }
         },
         'toggle-toolbar':{
diff --git a/notebook/static/notebook/js/cell.js b/notebook/static/notebook/js/cell.js
index d94501389..9b6379a68 100644
--- a/notebook/static/notebook/js/cell.js
+++ b/notebook/static/notebook/js/cell.js
@@ -14,8 +14,9 @@ define([
     'codemirror/lib/codemirror',
     'codemirror/addon/edit/matchbrackets',
     'codemirror/addon/edit/closebrackets',
-    'codemirror/addon/comment/comment'
-], function(utils, CodeMirror, cm_match, cm_closeb, cm_comment) {
+    'codemirror/addon/comment/comment',
+    'services/config',
+], function(utils, CodeMirror, cm_match, cm_closeb, cm_comment, configmod) {
     "use strict";
     
     var overlayHack = CodeMirror.scrollbarModel.native.prototype.overlayHack;
@@ -87,7 +88,12 @@ define([
         if(this.class_config){
             _local_cm_config = this.class_config.get_sync('cm_config');
         }
-        config.cm_config = utils.mergeopt({}, config.cm_config, _local_cm_config);
+        
+        var local = new configmod.ConfigWithDefaults(options.config,
+            {}, 'Cell');
+        var llcm = local.get_sync('cm_config');
+
+        config.cm_config = utils.mergeopt({}, config.cm_config, utils.mergeopt({}, llcm, _local_cm_config));
         this.cell_id = utils.uuid();
         this._options = config;
 
diff --git a/notebook/static/notebook/js/codecell.js b/notebook/static/notebook/js/codecell.js
index 6be236899..ef69791c1 100644
--- a/notebook/static/notebook/js/codecell.js
+++ b/notebook/static/notebook/js/codecell.js
@@ -108,7 +108,7 @@ define([
         this.completer = null;
 
         Cell.apply(this,[{
-            config: $.extend({}, CodeCell.options_default), 
+            config: options.config, 
             keyboard_manager: options.keyboard_manager, 
             events: this.events}]);
 
diff --git a/notebook/static/notebook/js/notebook.js b/notebook/static/notebook/js/notebook.js
index 54fe807bb..cad53475e 100644
--- a/notebook/static/notebook/js/notebook.js
+++ b/notebook/static/notebook/js/notebook.js
@@ -65,7 +65,6 @@ import {ShortcutEditor} from 'notebook/js/shortcuteditor';
         this.ws_url = options.ws_url;
         this._session_starting = false;
         this.last_modified = null;
-        this.line_numbers = false;
         // debug 484
         this._last_modified = 'init';
         // Firefox workaround
@@ -162,11 +161,28 @@ import {ShortcutEditor} from 'notebook/js/shortcuteditor';
         slideshow_celltoolbar.register(this);
         attachments_celltoolbar.register(this);
 
+        var that = this;
+
+        Object.defineProperty(this, 'line_numbers', {
+          get: function(){
+            var d = that.config.data||{}
+            var cmc =  (d['Cell']||{})['cm_config']||{}
+            return cmc['lineNumbers'] || false;
+          },
+          set: function(value){
+            this.get_cells().map(function(c) {
+              c.code_mirror.setOption('lineNumbers', value);
+            })
+            that.config.update({'Cell':{'cm_config':{'lineNumbers':value}}})
+          }
+          
+        })
         // prevent assign to miss-typed properties.
         Object.seal(this);
     };
 
 
+
     Notebook.options_default = {
         // can be any cell type, or the special values of
         // 'above', 'below', or 'selected' to get the value from another cell.
@@ -564,10 +580,6 @@ import {ShortcutEditor} from 'notebook/js/shortcuteditor';
      */
     Notebook.prototype.toggle_all_line_numbers = function () {
         this.line_numbers = !this.line_numbers;
-        var display = this.line_numbers;
-        this.get_cells().map(function(c) {
-            c.code_mirror.setOption('lineNumbers', display);
-        })
     }
 
     /**
diff --git a/notebook/static/notebook/js/textcell.js b/notebook/static/notebook/js/textcell.js
index abb6d9b1d..9ba54b4ae 100644
--- a/notebook/static/notebook/js/textcell.js
+++ b/notebook/static/notebook/js/textcell.js
@@ -58,7 +58,7 @@ define([
         // we cannot put this as a class key as it has handle to "this".
         var config = utils.mergeopt(TextCell, this.config);
         Cell.apply(this, [{
-                    config: config, 
+                    config: options.config, 
                     keyboard_manager: options.keyboard_manager, 
                     events: this.events}]);
 
@@ -282,7 +282,7 @@ define([
         var config = utils.mergeopt(MarkdownCell, {});
         this.class_config = new configmod.ConfigWithDefaults(options.config,
                                             {}, 'MarkdownCell');
-        TextCell.apply(this, [$.extend({}, options, {config: config})]);
+        TextCell.apply(this, [$.extend({}, options, {config: options.config})]);
 
         this.cell_type = 'markdown';
 
@@ -528,7 +528,7 @@ define([
          */
         options = options || {};
         var config = utils.mergeopt(RawCell, {});
-        TextCell.apply(this, [$.extend({}, options, {config: config})]);
+        TextCell.apply(this, [$.extend({}, options, {config: options.config})]);
 
         this.class_config = new configmod.ConfigWithDefaults(options.config,
                                             RawCell.config_defaults, 'RawCell');