Opened 12 years ago
Closed 2 years ago
#9512 closed enhancement (invalid)
Sage Source Editor
| Reported by: | Alex Leone | Owned by: | jason, was |
|---|---|---|---|
| Priority: | minor | Milestone: | sage-duplicate/invalid/wontfix |
| Component: | notebook | Keywords: | |
| Cc: | Kelly Boothby, Alex Leone, Carl Witty, Frédéric Chapoton | Merged in: | |
| Authors: | Tom Boothby | Reviewers: | Alex Leone |
| Report Upstream: | N/A | Work issues: | |
| Branch: | Commit: | ||
| Dependencies: | Stopgaps: |
Status badges
Description
Add an "Edit this page" link at the bottom of source files (/src/...) where one can edit the file.
Only user_type(username) == 'admin' can edit the files.
Attachments (1)
Change History (15)
Changed 12 years ago by
| Attachment: | 9512_source_editor.patch added |
|---|
comment:1 Changed 12 years ago by
| Status: | new → needs_review |
|---|
comment:2 Changed 12 years ago by
| Reviewers: | → Alex Leone |
|---|---|
| Status: | needs_review → positive_review |
comment:3 Changed 12 years ago by
comment:4 Changed 12 years ago by
| Merged in: | → sagenb-0.8.2 |
|---|---|
| Resolution: | → fixed |
| Status: | positive_review → closed |
comment:5 Changed 12 years ago by
| Merged in: | sagenb-0.8.2 |
|---|---|
| Resolution: | fixed |
| Status: | closed → new |
Actually, I found a bug: the "source editor" feature (#9512) converts line endings from Unix to DOS (so once you've edited the file, mercurial thinks every line has changed).
Given the total non-discoverability of #9512, I'm not sure this bug is worth holding up the new spkg; I'll let somebody else decide that.
I also see this behavior. I'm reopening this ticket, changing its status to "needs work," and removing it, for now, from #9572's SageNB 0.8.2.
comment:6 Changed 12 years ago by
| Status: | new → needs_work |
|---|
comment:7 Changed 12 years ago by
| Priority: | trivial → minor |
|---|
comment:8 Changed 12 years ago by
| Cc: | Carl Witty added |
|---|
comment:9 Changed 9 years ago by
| Milestone: | sage-5.11 → sage-5.12 |
|---|
comment:10 Changed 9 years ago by
| Milestone: | sage-6.1 → sage-6.2 |
|---|
comment:11 Changed 9 years ago by
| Milestone: | sage-6.2 → sage-6.3 |
|---|
comment:12 Changed 8 years ago by
| Milestone: | sage-6.3 → sage-6.4 |
|---|
comment:13 Changed 2 years ago by
| Cc: | Frédéric Chapoton added |
|---|---|
| Milestone: | sage-6.4 → sage-duplicate/invalid/wontfix |
| Status: | needs_work → needs_review |
Proposing to close all sagenb tickets as outdated, so that all remaining open tickets in the notebook component are about the Jupyter notebook.
comment:14 Changed 2 years ago by
| Resolution: | → invalid |
|---|---|
| Status: | needs_review → closed |
I'm added the commit string
for the patch I'm merging into SageNB 0.8.2 (#9572).