Skip to content

Monkey patch 102516 #2

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

Open
wants to merge 475 commits into
base: master
Choose a base branch
from
Open

Monkey patch 102516 #2

wants to merge 475 commits into from

Conversation

dalejung
Copy link
Owner

Really two issues.

  1. Make the main.js webpack modules accessible from custom.js. I did this by just creating an export module that modifies the global IPython object.
  2. The timing of custom.js. Previously, custom.js would be fully loaded before the notebook loaded. Since the notebook is now webpacked, custom.js loads async as the notebook is rendering. So you might monkey patch the correct objects, but the bind_events has already referenced the old function. I pushed the loading to a function that is called after custom.js.

Also, note that I had to change nbextension loading to never Promise.reject and to handle errors internally. Promise.all fails fast which means if a single nbextension 404s then other nbextensions get loaded after the notebook get loaded.

@dalejung dalejung mentioned this pull request Oct 30, 2016
ivanov and others added 29 commits January 24, 2017 16:50
along with a minor whitespace inconsistency introduced in an earlier
commit.
Add "Shutdown" item to Kernel menu
typo introduced in unwebpackification
* Edit notebook opens a notebook, Edit any other file type open it in a
text editor
* View only available for HTML files to disambiguate between “viewing”
a notebook or text file (which is the same as editing it) and viewing
an HTML file
without this, it is not possible to select any text using the mouse in
the contents of the modal. For example: editing the notebook metadata
JSON you end up just moving the whole box around instead of being able
to select a portion of the JSON using the mouse.
drag modal dialogs by the header only
When a kernelspec is removed while a kernel spawned from it is in use
the Running tab on the dashboards page errors and no longer shows
any results. Fall back on using the kernel name from the session
in this case so that users can see and terminate kernels.
Welcome Message for Console and Terminal Output
Carreau and others added 24 commits March 16, 2017 17:02
Prevent making releases with wrong version number that will confuse pip,
or make sdists to appear more recent than wheels.
Enforce version number at build time.
remove reference to `nodejs-legacy`, since apt packages are usually too old, anyway
make `--user` the non-default option,
given the prevalence of conda/virtual envs these days
minor updates to CONTRIBUTING installation instructions
This makes the high-res screenshots larger on a lower-res screen, but
nbsphinx doesn't currently handle HTML <img> tags, so we need to use the
Markdown format to show images in our docs.

Closes jupytergh-2304
Use Markdown image references in example notebook
Move preact, preact-compat, and proptypes to bower
Likely a side-effect of using `.form-control()`
Remove drop shadow from Tag UI for consistency.
Explain how to use cell tags in changelog
only consider stream output for data rate limit
…rate

Revert "only consider stream output for data rate limit"
@dalejung dalejung force-pushed the monkey_patch_102516 branch from 65b9fc9 to ffaf67c Compare March 25, 2017 17:54
before notebook itself loads. Previously a single nbextension loading
error would mess up loading since Promise.all fails fast.
dalejung pushed a commit that referenced this pull request Jun 24, 2023
* Add Binder

* Build from source on Binder

* Add support for notebook server extensions

* Add notebook shim

* Remove python from deps

* Add templates to MANIFEST

* Binder

* Add matplotlib and numpy to Binder
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.