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
Clean up notebook Javascript #2828
Conversation
This completely decouples the connection failed event in kernel.js from its handling in notificationarea.js.
var ws_url = data.ws_url; | ||
var early = data.early; | ||
var msg; | ||
console.log(early); |
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.
debug msg?
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.
Fixed.
I have also cleaned up some calls to console.log.
* Removed Tooltip._cmfocus as it was doing nothing. * Removed debug console.log in notification area.
While you are refactoring. Unless you know a such a lib... |
What about |
Otherwise, ok for me. |
@Carreau I think for now I am going to leave it as cell, because tooltip only works with codecells. If the MD cells become more kernel aware, we can fix that then. Merging. |
Clean up notebook Javascript
Clean up notebook Javascript
There were a number of spots where the CodeCell logic depended on the IPython.notebook object. These have been removed. I have also cleaned up some of the css that was having problems when CodeCells are embedded onto bootstrap containing pages. This will help us as we refactor the notebook to use bootstrap. Finally I have further decoupled the Kernel object from everything else and improved its reconnection logic.