-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
Update to CodeMirror 3 and start to ship our components #3232
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
Merged
Merged
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
a0cc2fb
Bower has changed component.json to bower.json. Complying.
ellisonbg 6339028
Updating bower.json for our CodeMirror repo.
ellisonbg 9b11915
Removing static/components from .gitignore-we want to ship this.
ellisonbg 647f4b0
Adding our current components directory.
ellisonbg 2571b05
Removing tests from fabfile that are no longer needed.
ellisonbg dfb847d
Moving CodeMirror -> cmtemp
ellisonbg f6bdcd1
Moving cmtemp -> codemirror.
ellisonbg 5590af4
Updating bower.json files.
ellisonbg 40537f2
Updating path to codemirror.
ellisonbg 4728c84
Updating tooltip.js to work with CM3.
ellisonbg f398099
In CM3, the class that needs autogrow has changed.
ellisonbg b978375
Updating tooltip.less for CM3 behavior.
ellisonbg e2e3fd3
Add components to static URLs.
ellisonbg da0782d
Updating style.min.css.
ellisonbg 66de4c5
Copying CodeMirror's python mode into place for our ipython mode.
ellisonbg 22418c9
Updating ipython CM mode.
ellisonbg 688001c
Fixing typo in ntoebook.html.
ellisonbg a0c4e41
Changing mode name from python -> ipython.
ellisonbg 61a613e
Fixing broken URLs to CM assets.
ellisonbg 5dafbbd
Give the .CodeMirror class no background so ours shows through.
ellisonbg f2bc065
Fixing CM3 style related issues.
ellisonbg d077f91
Removing old codemirror assets - we now use the 1 in components.
ellisonbg 198b5d5
Fix CM related sizing issues for completer and tooltip.
ellisonbg 7d3938c
Put back the components fab function as we can still use it.
ellisonbg 3940682
Adding README.md about notebook development.
ellisonbg 77d9286
Fixing indentation in fabfile.py.
ellisonbg File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,29 @@ | ||
# IPython Notebook development | ||
|
||
# Development dependencies | ||
|
||
Developers of the IPython Notebook will need to install the following tools: | ||
|
||
* fabric | ||
* node.js | ||
* less (`npm install -g less`) | ||
* bower (`npm install -g bower`) | ||
|
||
# Components | ||
|
||
We are moving to a model where our JavaScript dependencies are managed using | ||
[bower](http://bower.io/). These packages are installed in `static/components` | ||
and commited into our git repo. Our dependencies are described in the file | ||
`static/bower.json`. To update our bower packages, run `fab components` in this | ||
directory. | ||
|
||
Because CodeMirror does not use proper semantic versioning for its GitHub tags, | ||
we maintain our own fork of CodeMirror that is used with bower. This fork should | ||
track the upstream CodeMirror exactly; the only difference is that we are adding | ||
semantic versioned tags to our repo. | ||
|
||
# less | ||
|
||
If you edit our `.less` files you will need to run the less compiler to build | ||
our minified css files. This can be done by running `fab css` from this directory. | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,9 @@ | ||
{ | ||
"name": "ipython-notebook", | ||
"version": "1.0.0", | ||
"dependencies": { | ||
"bootstrap": "2.3.1", | ||
"less.js": "1.3.3", | ||
"codemirror": "git://github.com/ipython/CodeMirror.git#3.12.0" | ||
} | ||
} |
58 changes: 0 additions & 58 deletions
58
IPython/frontend/html/notebook/static/codemirror/README-IPython.rst
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why remove the components command? Won't we still want to use this when we update the versions we use?