o does nothing on the repo list, Repo box and ★ By repo rows #117

New issue

deadstyle · 2026-10-10T07:21:10-05:00

deadstyle commented

2026-10-10 07:21:10 -05:00

Problem

PM report: "the open(o) command isn't working for forgejo. Click it and get nothing."

Reproduced (2026-10-10, Forgejo host): o opens PRs, issues, runs, releases, branches and files correctly (the system opener received the right URL and opened a tab). It silently does nothing wherever the selection is a repo itself, because the open-target lookup has no case for a repo, so the key is disabled there:

the repo list
the [6] Repo box (README tab)
★ Renovate [1] By repo rows

Acceptance criteria

o on the repo list opens the selected repo's page in the browser (PM, 2026-10-10)
o on the Repo box README tab opens the repo's page (design.md → "o opens the browser for anything lazyforge doesn't cover")
o on a ★ By repo row opens that repo's page (same)
o shows in the repo-list key hints and help where it's enabled
Works the same on Forgejo/Gitea and GitHub

Context

internal/ui/actions.go webURL handles CR, Issue, Run, Release, Branch, TreeEntry but not domain.Repo (which carries WebURL). boxes.selected() returns b.repoRow (domain.Repo) for the Repo box.
Repo-level keys: syncKeys nils the item at levelRepos; repo-level short help doesn't list o.
★ starModel.item() returns nil for By repo rows; the repo's web URL needs to come from the repo list.
design.md keymap table and "Action keys … work at both the box and details levels" (line 31) need a note that o also works on the repo list.

## Problem PM report: "the open(o) command isn't working for forgejo. Click it and get nothing." Reproduced (2026-10-10, Forgejo host): `o` opens PRs, issues, runs, releases, branches and files correctly (the system opener received the right URL and opened a tab). It silently does nothing wherever the selection is a repo itself, because the open-target lookup has no case for a repo, so the key is disabled there: - the repo list - the `[6]` Repo box (README tab) - ★ Renovate `[1]` By repo rows ## Acceptance criteria - `o` on the repo list opens the selected repo's page in the browser *(PM, 2026-10-10)* - `o` on the Repo box README tab opens the repo's page *(design.md → "`o` opens the browser for anything lazyforge doesn't cover")* - `o` on a ★ By repo row opens that repo's page *(same)* - `o` shows in the repo-list key hints and help where it's enabled - Works the same on Forgejo/Gitea and GitHub ## Context - `internal/ui/actions.go` `webURL` handles CR, Issue, Run, Release, Branch, TreeEntry but not `domain.Repo` (which carries `WebURL`). `boxes.selected()` returns `b.repoRow` (`domain.Repo`) for the Repo box. - Repo-level keys: `syncKeys` nils the item at `levelRepos`; repo-level short help doesn't list `o`. - ★ `starModel.item()` returns nil for By repo rows; the repo's web URL needs to come from the repo list. - design.md keymap table and "Action keys … work at both the box and details levels" (line 31) need a note that `o` also works on the repo list.

deadstyle added the

bug

sonnet

labels

2026-10-10 07:21:12 -05:00

deadstyle referenced this issue from a commit

2026-10-10 07:24:49 -05:00

fix(ui): open the repo's page from the repo list, Repo box and ★ By repo

deadstyle referenced this issue from a commit

2026-10-10 07:25:42 -05:00

docs(ui): say why the By repo row needs its own open target

deadstyle referenced this issue