Update actor sidebar metadata

erik commented

2026-06-22 12:43:21 -05:00

Owner

Copy link

Goal

Improve sidebar actor cards and navigation sections across feeds, sources, accounts, tags, and articles.

Requirements

Show follower and following counts on the actor card when an actor is configured.
Do not show follower or following counts when no actor is configured.
Remove the duplicated “Browse Sources” link that also appears as “Recommended Site”.
Move “Browse Tags” into its own section.
Remove the unnecessary RSS section.

Acceptance criteria

Actor cards show follower and following counts only when actor data is configured.
Feeds, sources, accounts, tags, and articles sidebars use the updated actor card behavior.
The duplicated sources/recommended site link is resolved.
Browse tags appears in its own section.
The RSS section is removed.

Dependencies

None

## Goal Improve sidebar actor cards and navigation sections across feeds, sources, accounts, tags, and articles. ## Requirements - Show follower and following counts on the actor card when an actor is configured. - Do not show follower or following counts when no actor is configured. - Remove the duplicated “Browse Sources” link that also appears as “Recommended Site”. - Move “Browse Tags” into its own section. - Remove the unnecessary RSS section. ## Acceptance criteria - [ ] Actor cards show follower and following counts only when actor data is configured. - [ ] Feeds, sources, accounts, tags, and articles sidebars use the updated actor card behavior. - [ ] The duplicated sources/recommended site link is resolved. - [ ] Browse tags appears in its own section. - [ ] The RSS section is removed. ## Dependencies - None

erik added the

priority:medium

status:inprogress

labels

2026-06-22 12:43:21 -05:00

erik commented

2026-06-22 12:54:25 -05:00

Author

Owner

Copy link

Synced from todu comment by @todu on 2026-06-22T17:52:25.818Z

PR Review

Summary: Approved for PR #193. Inspected task requirements, PR diff, PR-body local verification evidence (vitest public route tests, ./scripts/pre-pr.sh, npm run build && npm run lint && npm test), and Forgejo Actions #266 success (build-lint-test).

Acceptance criteria:

Actor cards show follower and following counts only when actor data is configured.
Feeds, sources, accounts, tags, and articles sidebars use the updated actor card behavior.
The duplicated sources/recommended site link is resolved.
Browse tags appears in its own section.
The RSS section is removed.

Blocking issues: none.
Warnings: none.

Final verdict: approved; awaiting human merge approval.

_Synced from todu comment by @todu on 2026-06-22T17:52:25.818Z_ ## PR Review Summary: Approved for PR #193. Inspected task requirements, PR diff, PR-body local verification evidence (`vitest` public route tests, `./scripts/pre-pr.sh`, `npm run build && npm run lint && npm test`), and Forgejo Actions #266 success (`build-lint-test`). Acceptance criteria: - [x] Actor cards show follower and following counts only when actor data is configured. - [x] Feeds, sources, accounts, tags, and articles sidebars use the updated actor card behavior. - [x] The duplicated sources/recommended site link is resolved. - [x] Browse tags appears in its own section. - [x] The RSS section is removed. Blocking issues: none. Warnings: none. Final verdict: approved; awaiting human merge approval.

erik

2026-06-22 13:27:26 -05:00

closed this issue
added
status:done
and removed
status:inprogress
labels

erik commented

2026-06-22 13:27:26 -05:00

Author

Owner

Copy link

Synced from todu comment by @todu on 2026-06-22T18:23:04.283Z

Closing Summary

PR #193 was merged to main as merge commit f1f2519 (Update actor sidebar metadata), with implementation commit cc1c0f3 for task task-5bc8415a.

Acceptance criteria verified as met:

Actor cards show follower/following counts only when actor data is configured.
Feeds, sources, accounts, tags, and articles sidebars use the updated actor card behavior.
The duplicated sources/recommended site link is resolved.
Browse tags appears in its own section.
The RSS section is removed.

Evidence: template/site/src/templates/public.tsx, template/site/src/routes/home.ts, and template/site/src/routes/__tests__/public.test.ts in merged PR #193.

_Synced from todu comment by @todu on 2026-06-22T18:23:04.283Z_ ## Closing Summary PR #193 was merged to `main` as merge commit `f1f2519` (`Update actor sidebar metadata`), with implementation commit `cc1c0f3` for task `task-5bc8415a`. Acceptance criteria verified as met: - Actor cards show follower/following counts only when actor data is configured. - Feeds, sources, accounts, tags, and articles sidebars use the updated actor card behavior. - The duplicated sources/recommended site link is resolved. - Browse tags appears in its own section. - The RSS section is removed. Evidence: `template/site/src/templates/public.tsx`, `template/site/src/routes/home.ts`, and `template/site/src/routes/__tests__/public.test.ts` in merged PR #193.

No Branch/Tag specified