Fix narrow article content width #188

Merged

erik merged 1 commit from fix/task-83b6b57c-article-content-width into main

2026-06-22 07:16:03 -05:00

erik commented

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

Owner

Summary

Fix post detail pages so article body content uses the available article card width instead of being capped at max-w-3xl.

Task

Task: #task-83b6b57c

Changes

Removed the max-w-3xl cap from post detail .post-content.
Added a public page regression assertion that post detail content no longer renders with the capped width class.

Testing

Unit tests added/updated
Manual testing performed

Verification:

npm exec --workspace @slugkit/template-site -- vitest run --config ../../vitest.config.ts src/routes/__tests__/public.test.ts
Chromium inspection at /posts/decentralized-social-media: article width 1120px, content width 1040px, content max-width none.
./scripts/pre-pr.sh

Checklist

./scripts/pre-pr.sh passes
Documentation updated (if needed)
No unrelated changes included

## Summary Fix post detail pages so article body content uses the available article card width instead of being capped at `max-w-3xl`. ## Task Task: #task-83b6b57c ## Changes - Removed the `max-w-3xl` cap from post detail `.post-content`. - Added a public page regression assertion that post detail content no longer renders with the capped width class. ## Testing - [x] Unit tests added/updated - [x] Manual testing performed Verification: - `npm exec --workspace @slugkit/template-site -- vitest run --config ../../vitest.config.ts src/routes/__tests__/public.test.ts` - Chromium inspection at `/posts/decentralized-social-media`: article width `1120px`, content width `1040px`, content max-width `none`. - `./scripts/pre-pr.sh` ## Checklist - [x] `./scripts/pre-pr.sh` passes - [x] Documentation updated (if needed) - [x] No unrelated changes included

erik added 1 commit

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

fix: expand post detail content width

CI / build-lint-test (pull_request) Successful in 45s

Details

bffdfba9d3

Task: #task-83b6b57c

erik commented

2026-06-22 07:15:24 -05:00

Author

Owner

Review summary

PR #188 removes the max-w-3xl constraint from post detail .post-content, allowing the body to use the available article card width while preserving existing typography and markdown styling.

Acceptance criteria

Post detail content uses the available article content width at desktop sizes — max-w-3xl was removed from the post content container.
The layout remains readable and responsive at narrower viewport sizes — the surrounding article padding/responsive container remains unchanged, so narrow viewport behavior is still governed by the existing card/container layout.
Existing public page styling/tests are updated if needed — the public route regression test now asserts the uncapped class and rejects the old capped class.

Verification

Diff reviewed against task task-83b6b57c.
Local ./scripts/pre-pr.sh passed.
Forgejo Actions run #258 is success for build-lint-test on commit bffdfba.

Blocking issues

None.

Warnings

None.

Final verdict

Approved. Do not merge until explicit human merge approval is given.

## Review summary PR #188 removes the `max-w-3xl` constraint from post detail `.post-content`, allowing the body to use the available article card width while preserving existing typography and markdown styling. ## Acceptance criteria - [x] Post detail content uses the available article content width at desktop sizes — `max-w-3xl` was removed from the post content container. - [x] The layout remains readable and responsive at narrower viewport sizes — the surrounding article padding/responsive container remains unchanged, so narrow viewport behavior is still governed by the existing card/container layout. - [x] Existing public page styling/tests are updated if needed — the public route regression test now asserts the uncapped class and rejects the old capped class. ## Verification - Diff reviewed against task `task-83b6b57c`. - Local `./scripts/pre-pr.sh` passed. - Forgejo Actions run #258 is `success` for `build-lint-test` on commit `bffdfba`. ## Blocking issues None. ## Warnings None. ## Final verdict Approved. Do not merge until explicit human merge approval is given.

erik merged commit 033b09c657 into main

2026-06-22 07:16:03 -05:00

erik deleted branch fix/task-83b6b57c-article-content-width

2026-06-22 07:16:04 -05:00

erik referenced this pull request from a commit

2026-06-22 07:16:05 -05:00

Merge pull request 'Fix narrow article content width' (#188) from fix/task-83b6b57c-article-content-width into main

erik referenced this pull request

2026-06-22 07:18:14 -05:00

Fix narrow article content width #187

erik referenced this pull request

2026-06-22 07:18:14 -05:00