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
Add Edit Notebook Metadata to Edit menu #4333
Conversation
return false; | ||
} | ||
callback(new_md); | ||
}}, |
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.
Split }}
in two lines for better readability?
Just two minor comments... great to have UI to notebook metadata (not only cells). |
yup, style comments addressed (that code was just copied/pasted from celltoolbar, I didn't look too carefully at it). |
I notice that, this is why I added the comments, because is easy to fix them now ;-) |
And a perfect opportunity for review - thanks for the watchful eye! |
|
||
Notebook.prototype.edit_metadata = function () { | ||
var that = this; | ||
console.log('before', this.metadata); |
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.
forgotten debug ?
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.
@minrk ?
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.
yup, sorry. Fixed.
+1 |
Great, merging from my phone :-) (awesome new feature) |
Add Edit Notebook Metadata to Edit menu
Add Edit Notebook Metadata to Edit menu
Moved the raw-edit dialog to IPython.dialog so it can be reused.