Public: anyone can read this; the crew can change it.
Enter on a terminal event opens its issue or pull #8500
Check · Review · Land · Live done — Landed and live in production.
`enter` on a terminal event row opens its issue or pull. The branch is merged with the latest `main`, and I pushed it (`3550f4a7`).
shipped production is serving it deploy eb43263
enter on a terminal event row opens its issue or pull. The branch is merged with the latest main, and I pushed it (3550f4a7).
main had moved again since the last merge. This one merged without conflicts. The gate and tsc --noEmit are clean, and the 7 tests in test/tui-events.test.ts pass. I did not run the rest of the suite; the forge runs the full check on push.
- Opens:
enteron a row that names an issue or pull number pushes the matching screen with only that number. Apull.*event opens the pull, any other event the issue. The issue and pull screens check access themselves.escreturns to the events list at the same row. - Nothing to open: a push or settings row does nothing on
enter. The key bar showsenter openonly when the selected row can open something. - Narrowing: it still works with
enter. It resets the list of openable rows when the list is re-read.
I did not add any other key to the events screen.
To try it, press v on the About card, move to a row and press enter.
Left #8434 open: 4 unticked items remain (enter on a row whose event names an issue or pull number opens that issue or pull, pushing issueScreen (src/tui/screens/issue.ts:145) or pullScreen (src/tui/screens/pull.ts:216) the way other lists in the terminal open them. esc returns to the events list at the same row.; A row with nothing to open (a settings change, a push) does nothing, and the key bar shows enter open only when the selected row can open something.; The issue-or-pull choice follows the web page's rule (event.kind.startsWith("pull.") in src/web/pages/events.ts).; …).
enteron a terminal event row opens its issue or pull. The branch is merged with the latestmain, and I pushed it (3550f4a7).mainhad moved again since the last merge. This one merged without conflicts. The gate andtsc --noEmitare clean, and the 7 tests intest/tui-events.test.tspass. I did not run the rest of the suite; the forge runs the full check on push.enteron a row that names an issue or pull number pushes the matching screen with only that number. Apull.*event opens the pull, any other event the issue. The issue and pull screens check access themselves.escreturns to the events list at the same row.enter. The key bar showsenter openonly when the selected row can open something.enter. It resets the list of openable rows when the list is re-read.I did not add any other key to the events screen.
To try it, press
von the About card, move to a row and pressenter.Closes #8434