4 files changed,
+18
−6
internal/sheets/handlers_list_chrome_test.go
+1 −1
internal/sheets/handlers_sheets.go
+1 −1
internal/sheets/templates/layout.html
+10 −3
internal/sheets/templates/sheets.html
+6 −1
Intent
Attributed to this commit: 45.8 g CO₂e (4587742 tokens). This is an attribution, not a measurement — usage counted from this commit's own marker forward to the next one on worktree-list-chrome-rebase, never backward from it. A turn that produced more than one commit books all of its tokens to the last one, and a concurrent session on this branch can add to the same figure. Some of it was priced using the shared default rate, because no model-specific factor row exists yet for at least one model involved. The Carbon view has the repo total, which is never derived from this attribution.
full ledger: worktree-list-chrome-rebase
| index 8282dee..5369f48 100644 |
| --- a/internal/sheets/handlers_list_chrome_test.go |
| +++ b/internal/sheets/handlers_list_chrome_test.go |
| @@ -210,7 +210,7 @@ func TestRefusalRendersOnTheImportPage(t *testing.T) { |
| if !strings.Contains(body, `action="/sheets/import"`) { |
| t.Error("the refusal rendered somewhere without the form to retry from") |
| } |
| - if strings.Contains(body, "Your sheets</h1>") { |
| + if strings.Contains(body, "All sheets</h1>") { |
| t.Error("the refusal rendered on the sheet list") |
| } |
| } |
| index c4f6b3c..2c79762 100644 |
| --- a/internal/sheets/handlers_sheets.go |
| +++ b/internal/sheets/handlers_sheets.go |
| @@ -104,7 +104,7 @@ func (a *App) renderSheetList(w http.ResponseWriter, r *http.Request, status int |
| for _, s := range list { |
| rows = append(rows, sheetRow{SheetSummary: s, Preview: a.previewFor(s)}) |
| } |
| - a.render(w, r, "sheets", "Your sheets — Oficina Sheets", status, |
| + a.render(w, r, "sheets", "All sheets — Oficina Sheets", status, |
| sheetListView{Sheets: rows}) |
| } |
| |
| index 77ed0ee..9ab26a8 100644 |
| --- a/internal/sheets/templates/layout.html |
| +++ b/internal/sheets/templates/layout.html |
| @@ -75,9 +75,16 @@ |
| </details> |
| {{end}} |
| <div rst-shell-tail> |
| -<nav rst-shell-nav> |
| -{{if .SignedIn}}<a href="/sheets">Your sheets</a>{{end}} |
| -</nav> |
| +{{/* There is no nav here, and the empty landmark went with the link. |
| + |
| + It held one item, "Your sheets", pointing at /sheets — which is |
| + where the brand two elements to the left already goes: handleIndex |
| + 303s a signed-in visitor from / to /sheets. So the bar read |
| + "Sheets ... Your sheets" and both were the same destination under |
| + two names, which is a menu of one choice offered twice. |
| + |
| + The brand is the way back to the list. That is what a wordmark in |
| + the top-left is for, and it is one fewer thing to read. */}} |
| {{/* A page's own top-bar actions, beside the account menu. |
| |
| Deliberately a block rather than a field on PageData, and |
| index 78534f1..8269e9e 100644 |
| --- a/internal/sheets/templates/sheets.html |
| +++ b/internal/sheets/templates/sheets.html |
| @@ -51,7 +51,12 @@ |
| {{define "content"}} |
| <header rst-page-header> |
| <div rst-page-header-titles> |
| -<h1>Your sheets</h1> |
| +{{/* "All sheets", not "Your sheets". The old heading was wrong twice: |
| + it repeated the brand in the bar directly above it, and it claimed |
| + ownership the line underneath immediately contradicts — everyone on |
| + the instance opens every sheet here, so almost none of these are |
| + yours. A heading its own subtitle corrects is worse than none. */}} |
| +<h1>All sheets</h1> |
| <p rst-page-header-sub>Everyone on this instance can open every sheet here.</p> |
| </div> |
| </header> |