Setting the mode many times in a row, would fail to actually set the
mode if it was failing to load the mode the first time.
The prevent the above behavior by falling back **explicitly** to 'null'
if mode does not exist yet, or can't be loaded.
Also, don't set to null mode if mode have changed
Or if kernel.js have changed the mode, we re-set it to null...
- double-click toggles 'raw' size for large image output
- uses max-width on both markdown and output images
- `height: auto` fixes aspect ratio problems caused by using max-width alone
- img.unconfined disables max-width confinement
action might be undefined then button will get no icon.
Prevent throwing an uncatched error and insert warning sign on button
Will do better one I allow text instead of icon
When triggering actions with button one have to refocus the notebook
cell. Though with keyboard shortcut, you don't want cell to switch to
command mode.
code cell where actually not configurable,
this fixes that + add a convenient methods to dump the
content of the configuration in the config file.
this also close#7896 by defaulting closebracket to true,
which is now configurable :-)
It also switches the default magic highlight mode to string so that they
could also be configured.
The function in charge of actually converting cursor offset to
CodeMirror line number and character number was actually crashing when
the cursor was at the last character (loop until undefined, then access
length of variable, which is undefined).
This was hiding a bug in which when you would completer to a single
completion pressing tab after as-you-type filtering, the completion
would be completed twice.
The logic that was supposed to detect whether or not all completions had
a common prefix was actually faulty as the common prefix used to be a
string but was then changed to an object. Hence the logic to check
whether or not there was actually a common prefix was always true, even
for empty string, leading to the deletion of the line (replace by '') in
some cases.