Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some typos #5917

Merged
merged 1 commit into from
Dec 30, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion notebook/bundler/tests/test_bundler_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ def test_missing_bundler_arg(self):
self.assertIn('Missing argument bundler', resp.text)

def test_notebook_not_found(self):
"""Shoudl respond with 404 error about missing notebook"""
"""Should respond with 404 error about missing notebook"""
resp = self.request('GET', 'bundle/fake.ipynb',
params={'bundler': 'fake_bundler'})
self.assertEqual(resp.status_code, 404)
Expand Down
2 changes: 1 addition & 1 deletion notebook/static/notebook/js/celltoolbar.js
Original file line number Diff line number Diff line change
Expand Up @@ -403,7 +403,7 @@ define([
* @static
*
* @param list_list {list_of_sublist} List of sublist of metadata value and name in the dropdown list.
* subslit should contain 2 element each, first a string that woul be displayed in the dropdown list,
* sublist should contain 2 element each, first a string that would be displayed in the dropdown list,
* and second the corresponding value to be passed to setter/return by getter. the corresponding value
* should not be "undefined" or behavior can be unexpected.
* @param setter {function( cell, newValue )}
Expand Down
2 changes: 1 addition & 1 deletion notebook/static/notebook/js/codecell.js
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,7 @@ define([
}

if (event.which === keycodes.down && event.type === 'keypress' && this.tooltip.time_before_tooltip >= 0) {
// triger on keypress (!) otherwise inconsistent event.which depending on plateform
// triger on keypress (!) otherwise inconsistent event.which depending on platform
// browser and keyboard layout !
// Pressing '(' , request tooltip, don't forget to reappend it
// The second argument says to hide the tooltip if the docstring
Expand Down
4 changes: 2 additions & 2 deletions notebook/static/notebook/js/completer.js
Original file line number Diff line number Diff line change
Expand Up @@ -348,7 +348,7 @@ define([
} else if (code == keycodes.tab) {
//all the fastforwarding operation,
//Check that shared start is not null which can append with prefixed completion
// like %pylab , pylab have no shred start, and ff will result in py<tab><tab>
// like %pylab , pylab have no shared start, and ff will result in py<tab><tab>
// to erase py
var sh = shared_start(this.raw_result, true);
if (sh.str !== '') {
Expand All @@ -358,7 +358,7 @@ define([
this.carry_on_completion();
} else if (code == keycodes.up || code == keycodes.down) {
// need to do that to be able to move the arrow
// when on the first or last line ofo a code cell
// when on the first or last line of a code cell
event.codemirrorIgnore = true;
event._ipkmIgnore = true;
event.preventDefault();
Expand Down
4 changes: 2 additions & 2 deletions notebook/static/notebook/js/tooltip.js
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ define([
};

// deal with all the logic of hiding the tooltip
// and reset it's status
// and reset its status
Tooltip.prototype._hide = function () {
this._hidden = true;
this.tooltip.fadeOut('fast');
Expand Down Expand Up @@ -243,7 +243,7 @@ define([
this._sticky = false;
};

// put the tooltip in a sicky state for 10 seconds
// put the tooltip in a sticky state for 10 seconds
// it won't be removed by remove_and_cancel() unless you called with
// the first parameter set to true.
// remove_and_cancel_tooltip(true)
Expand Down