-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
True read-only cells #1402
Comments
Do you want it in the core? Or as an extension ? By read-only, I suppose you want users to not be able to clear output, and not modify the text of cell. |
Yes, I'd like it in core. Ideally students shouldn't have to install anything extra.
I would be ok with clearing output -- just not editing the source of the cell. I think not being able to clear the output (or overwrite it, if the cell is re-run) could be confusing.
I think they should probably be able to see the source of markdown, just not edit it.
I think disabling the cursor would be fine.
Yes, it should be copiable, and pasted versions shouldn't be read-only. I would expect the behavior to be similar to how the "undeletable" property is maintained (and actually, "read-only" should imply "undeleteable", I think). |
Is there even a way to enforce this 'serverside' at all? If not, just adding a 'readonly' field to cell metadata and enforcing it in client side JS (either in an extension or core) would be enough, right? |
(pressed wrong button, sorry!) |
No, it's something that the frontend would have to support. So yeah, adding a 'readonly' field to the cell metadata and then enforcing it in the core notebook client JS (this is also how the 'deletable' field works). |
Yes, if/when we document this, we should make it clear that it's a 'stop users shooting themselves in the foot' measure, not a security measure. |
nbgrader currently has pseudo "read-only" cells where they are "read-only" in the sense that their contents get saved to the nbgrader database, and then overwritten during grading if they have been changed by the student. I have reports that this is confusing, and requests for true read-only cells where the cell actually cannot be edited at all in the live notebook.
I think this would be really useful to have for nbgrader. We could keep it as an undocumented feature for the moment, just like "undeletable" cells are, until we can figure out an appropriate UI for it.
The text was updated successfully, but these errors were encountered: