forgejo_admin/pal-e-docs-app
1
0
Fork 0
Code Issues 5 Pull requests 2 Projects Releases Packages Wiki Activity Actions

Port: notes list page from playground #70

New issue
Closed
opened 2026-03-27 21:14:32 +00:00 by forgejo_admin · 3 comments
forgejo_admin commented 2026-03-27 21:14:32 +00:00
Owner
Copy link

Type

Feature

Lineage

Depends on #68 (CSS + sidebar foundation). Part of SvelteKit port pipeline.

Repo

forgejo_admin/pal-e-app

User Story

As a reader, I want a filterable notes list with type-colored cards, so I can browse and find notes by type.

Context

Restyle src/routes/notes/+page.svelte to match ~/pal-e-docs-playground/notes.html. The notes list page already exists with filtering — this is a restyle to match the playground design, not greenfield. Data fetching uses client-side onMount, not SvelteKit load functions — no +page.ts needed.

File Targets

  • src/routes/notes/+page.svelte — restyle existing notes list with filter bar and type-colored cards to match playground

Acceptance Criteria

  • Notes list layout matches playground notes.html
  • Filter bar functional (filter by note type)
  • Type-colored cards render correctly
  • App builds without errors

Test Expectations

  • Run command: cd ~/pal-e-app && npm run build

Constraints

  • Copy playground HTML literally, swap data bindings.
  • This is a restyle of an existing page, not a new route.
  • Depends on #68 being merged first.

Checklist

  • Page restyled
  • Filters still functional
  • Build passes

Related

  • forgejo_admin/pal-e-app#68 — foundation (dependency)
  • board-pal-e-docs — project board
### Type Feature ### Lineage Depends on #68 (CSS + sidebar foundation). Part of SvelteKit port pipeline. ### Repo `forgejo_admin/pal-e-app` ### User Story As a reader, I want a filterable notes list with type-colored cards, so I can browse and find notes by type. ### Context Restyle `src/routes/notes/+page.svelte` to match `~/pal-e-docs-playground/notes.html`. The notes list page already exists with filtering — this is a restyle to match the playground design, not greenfield. Data fetching uses client-side `onMount`, not SvelteKit load functions — no `+page.ts` needed. ### File Targets - `src/routes/notes/+page.svelte` — restyle existing notes list with filter bar and type-colored cards to match playground ### Acceptance Criteria - [ ] Notes list layout matches playground notes.html - [ ] Filter bar functional (filter by note type) - [ ] Type-colored cards render correctly - [ ] App builds without errors ### Test Expectations - Run command: `cd ~/pal-e-app && npm run build` ### Constraints - Copy playground HTML literally, swap data bindings. - This is a restyle of an existing page, not a new route. - Depends on #68 being merged first. ### Checklist - [ ] Page restyled - [ ] Filters still functional - [ ] Build passes ### Related - `forgejo_admin/pal-e-app#68` — foundation (dependency) - `board-pal-e-docs` — project board
forgejo_admin commented 2026-03-27 21:56:18 +00:00
Author
Owner
Copy link

Scope Review: NEEDS_REFINEMENT

Review note: review-472-2026-03-27

Two issues found:

  • src/routes/notes/+page.ts does not exist — app uses client-side fetching via onMount. Remove from file targets.
  • Page already exists with type-colored cards and filter bar. This is a restyle, not greenfield — ticket should note existing state.
## Scope Review: NEEDS_REFINEMENT Review note: `review-472-2026-03-27` Two issues found: - `src/routes/notes/+page.ts` does not exist — app uses client-side fetching via `onMount`. Remove from file targets. - Page already exists with type-colored cards and filter bar. This is a restyle, not greenfield — ticket should note existing state.
forgejo_admin commented 2026-03-27 22:06:10 +00:00
Author
Owner
Copy link

Issue body updated per scope review corrections.

Issue body updated per scope review corrections.
forgejo_admin commented 2026-03-28 04:56:47 +00:00
Author
Owner
Copy link

This issue was already implemented by PR #77 (commit 098260a, merged to main).

All acceptance criteria are met on current main:

  • Notes list layout matches playground notes.html (flat .note-card list with .note-card-row, badges, relative time)
  • Filter bar functional with dynamic type pills and URL-synced note_type param
  • Type-colored badge classes for all note types (badge--sop, badge--convention, badge--doc, badge--project, badge--board, etc.)
  • App builds without errors (npm run build passes)

Closing as already complete.

This issue was already implemented by PR #77 (commit 098260a, merged to main). All acceptance criteria are met on current main: - Notes list layout matches playground `notes.html` (flat `.note-card` list with `.note-card-row`, badges, relative time) - Filter bar functional with dynamic type pills and URL-synced `note_type` param - Type-colored badge classes for all note types (`badge--sop`, `badge--convention`, `badge--doc`, `badge--project`, `badge--board`, etc.) - App builds without errors (`npm run build` passes) Closing as already complete.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
92-fix-replace-plugin-git-clone-with-alpine
226-fix-wget-auth
No results found.
Labels
Clear labels   
domain:backend

   
domain:devops

   
domain:frontend
No labels
domain:backend
domain:devops
domain:frontend
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
forgejo_admin/pal-e-docs-app#70
No description provided.