Skip to content

Commit 446b9cb

Browse files
committed
Merge pull request jupyter#2 from svurens/typeahead
moved commandpalette to new class
2 parents e87561d + fe20457 commit 446b9cb

File tree

4 files changed

+49
-30
lines changed

4 files changed

+49
-30
lines changed

notebook/static/notebook/js/actions.js

Lines changed: 1 addition & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -299,35 +299,7 @@ define(function(require){
299299
'command-palette': {
300300
help_index : 'aa',
301301
handler : function(env){
302-
var form = $('<form/>');
303-
var container = $('<div/>').addClass('typeahead-container');
304-
var field = $('<div/>').addClass('typeahead-field');
305-
var span = $('<span>').addClass('typeahead-query');
306-
var input = $('<input/>').attr('type', 'search');
307-
308-
span.append(input)
309-
field.append(span)
310-
container.append(field)
311-
form.append(container)
312-
input.typeahead({
313-
order: "asc",
314-
source: {
315-
groupName: {
316-
data: [ 'california', 'washington', 'state', 'france', 'china', 'russia', 'rust', 'river' , 'repression']
317-
}
318-
},
319-
callback: {
320-
onInit: function () {console.log('this is init') }
321-
}
322-
})
323-
dialog.modal({
324-
title: 'Execute Action',
325-
body: $('<div/>').append(form),
326-
buttons: {
327-
OK: {'class': 'btn-primary'}
328-
},
329-
keyboard_manager: env.notebook.keyboard_manager
330-
});
302+
env.notebook.show_command_palette();
331303
}
332304
}
333305

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
// Copyright (c) Jupyter Development Team.
2+
// Distributed under the terms of the Modified BSD License.
3+
4+
define(function(require){
5+
"use strict";
6+
7+
var $ = require("jquery");
8+
var dialog = require("base/js/dialog");
9+
var CommandPalette = function(notebook) {
10+
var form = $('<form/>');
11+
var container = $('<div/>').addClass('typeahead-container');
12+
var field = $('<div/>').addClass('typeahead-field');
13+
var span = $('<span>').addClass('typeahead-query');
14+
var input = $('<input/>').attr('type', 'search');
15+
span.append(input)
16+
field.append(span)
17+
container.append(field)
18+
form.append(container)
19+
input.typeahead({
20+
order: "asc",
21+
source: {
22+
groupName: {
23+
data: [ 'california', 'washington', 'state', 'france', 'china', 'russia', 'rust', 'river' , 'repression']
24+
}
25+
},
26+
callback: {
27+
onInit: function () {console.log('this is init') }
28+
}
29+
})
30+
dialog.modal({
31+
title: 'Execute Action',
32+
body: $('<div/>').append(form),
33+
buttons: {
34+
OK: {'class': 'btn-primary'}
35+
},
36+
keyboard_manager: notebook.keyboard_manager
37+
});
38+
}
39+
return {'CommandPalette': CommandPalette};
40+
});
41+

notebook/static/notebook/js/keyboardmanager.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ define([
8484

8585
KeyboardManager.prototype.get_default_command_shortcuts = function() {
8686
return {
87-
'cmd-shift-P': 'ipython.command-palette',
87+
'shift-p': 'ipython.command-palette',
8888
'shift-space': 'ipython.scroll-up',
8989
'shift-v' : 'ipython.paste-cell-before',
9090
'shift-m' : 'ipython.merge-selected-cell-with-cell-after',

notebook/static/notebook/js/notebook.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ define(function (require) {
2727
var rawcell_celltoolbar = require('notebook/js/celltoolbarpresets/rawcell');
2828
var slideshow_celltoolbar = require('notebook/js/celltoolbarpresets/slideshow');
2929
var scrollmanager = require('notebook/js/scrollmanager');
30+
var commandpalette = require('notebook/js/commandpalette');
3031

3132
/**
3233
* Contains and manages cells.
@@ -319,6 +320,11 @@ define(function (require) {
319320
};
320321
};
321322

323+
324+
Notebook.prototype.show_command_palette = function() {
325+
var x = new commandpalette.CommandPalette(this);
326+
}
327+
322328
/**
323329
* Trigger a warning dialog about missing functionality from newer minor versions
324330
*/

0 commit comments

Comments
 (0)