Skip to content

Post page: put the link sections and comments on cards - #10

Merged
compscidr merged 1 commit into
mainfrom
fix/post-sections-on-card
Oct 3, 2026
Merged

compscidr merged 1 commit into
mainfrom
fix/post-sections-on-card

Conversation

@compscidr

Copy link
Copy Markdown
Contributor

Problem

On a post page, the admin-only link sections (Backlinks, Linked Posts, External Links, External Backlinks) and the comments were laid out directly on the background photo, outside the card the post body sits on. Dark text over the trees was hard to read.

Fix

  • post.html: the link sections share one glass-card, and the comments get another, both inside the same padded wrapper as the post body. The Bootstrap row/container wrappers around them are gone, since the card sets the width.
  • goblog.css: spacing between the link sections.
  • Changelog entry for 1.4.1. No change to min_goblog_version.

Testing

Rendered post.html (this same template, via the site theme) with sample data against goblog main, as admin and as a visitor, and checked a headless Chrome screenshot at 1300px wide: three cards, nothing on the bare background. Not checked on a phone-width screen or in a live GoBlog instance.

🤖 Generated with Claude Code

The admin-only link sections and the comments were laid out directly on
the background photo, outside the card the post body sits on, so their
text was hard to read. Each group now gets its own glass card.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

It is a small, self-contained presentational change with balanced, well-formed HTML across both render paths and a correct, idiomatic CSS addition, with no functional or logic changes.

Review effort: Balanced
Findings: None

What changed in this PR

This PR is a presentation-only change to the Forest theme's post page. Previously, the admin-only link sections (Backlinks, Linked Posts, External Links, External Backlinks) and the comments were rendered directly over the full-bleed forest background photo, making dark text hard to read. The change moves these into frosted glass-card containers (matching the post body) inside the same padded wrapper, replacing the removed Bootstrap row/container layout wrappers.

Changes:

  • post.html: wrap the four admin link sections in one glass-card and the comments in another, both inside the existing padded wrapper; drop the row/container wrappers and introduce post-links blocks.
  • goblog.css: add vertical spacing between consecutive .post-links sections.
  • CHANGELOG.md: add a 1.4.1 entry describing the fix.
File Description
templates/​post.html Places admin link sections and comments onto glass-card containers inside the post's padded wrapper; HTML remains balanced in both admin and visitor render paths.
static/​css/​goblog.css Adds .post-links + .post-links { margin-top: 24px; } to space adjacent admin link sections.
CHANGELOG.md Documents the 1.4.1 styling fix.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@compscidr
compscidr merged commit 4e7ccc1 into main Oct 3, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants