Hide-read choice resets between visits #49

New issue

Closed

opened 2026-09-25 23:49:20 +00:00 by agent-pi · 2 comments

agent-pi commented

2026-09-25 23:49:20 +00:00

Collaborator

The hide-read toggle on the reader currently works by adding ?hide_read=1 to URLs. The choice is lost as soon as you follow a link that does not carry the parameter, and it never survives closing the browser.

The preference should stick across navigation and across visits until it is toggled back off — a cookie is the natural fit for a server-rendered, JavaScript-free UI.

The hide-read toggle on the reader currently works by adding `?hide_read=1` to URLs. The choice is lost as soon as you follow a link that does not carry the parameter, and it never survives closing the browser. The preference should stick across navigation and across visits until it is toggled back off — a cookie is the natural fit for a server-rendered, JavaScript-free UI.

agent-pi referenced this issue

2026-09-25 23:49:38 +00:00

Reader cards show both group title and overview #48

agent-pi commented

2026-09-25 23:49:38 +00:00

Author

Collaborator

Plan of attack

Replace the hide_read query parameter entirely with a hide_read cookie read by the reader index and aggregate routes — no more threading the parameter through card links, form actions and redirects.
New POST /reader/hide-read toggle endpoint: flips the cookie (1 year max-age, HttpOnly, SameSite=Lax), redirects back to a validated next path (same-origin, no open redirect). Plain HTML form, no JavaScript.
Tests first: toggle sets/clears the cookie; index and aggregate pages filter by the cookie; mark-read redirects stay clean.

Lands together with #48, #50 and #41 on the shared branch feat/reader-tweaks-and-docs (one PR).

## Plan of attack - Replace the `hide_read` query parameter entirely with a `hide_read` cookie read by the reader index and aggregate routes — no more threading the parameter through card links, form actions and redirects. - New `POST /reader/hide-read` toggle endpoint: flips the cookie (1 year max-age, HttpOnly, SameSite=Lax), redirects back to a validated `next` path (same-origin, no open redirect). Plain HTML form, no JavaScript. - Tests first: toggle sets/clears the cookie; index and aggregate pages filter by the cookie; mark-read redirects stay clean. Lands together with #48, #50 and #41 on the shared branch `feat/reader-tweaks-and-docs` (one PR).

agent-pi referenced this issue

2026-09-25 23:49:39 +00:00

Footer links to the wrong source URL and hardcodes the version #50

agent-pi referenced this issue

2026-09-25 23:49:39 +00:00

Create documentation site for yunjin #41

agent-pi referenced this issue from a pull request that will close it,

2026-09-26 00:11:58 +00:00

Add documentation site and reader refinements #51

agent-pi closed this issue

2026-09-26 02:06:32 +00:00

agent-pi commented