RSS subscription + styled feed, and fix duplicate Pages deploy#2
Merged
Conversation
Enable the rss entry in the EN/LT social menus so the feed icon appears in the nav (github, ..., rss) and links to each language's home feed. RSS autodiscovery link and per-section feeds were already emitted by the theme. Co-authored-by: Cursor <cursoragent@cursor.com>
Attach an xml-stylesheet PI to the feed and ship static/feed.xsl so browsers render a styled, readable page (title, posts, subscribe hint) instead of raw XML, while feed readers keep consuming the underlying RSS. Note: Chromium removes native XSLT on 2026-11-17, after which browsers fall back to raw XML. Co-authored-by: Cursor <cursoragent@cursor.com>
Merging to main fired two competing Pages deployments — one from the push trigger and one from the Run Tests workflow_run hand-off — which queued in the pages concurrency group and raced, causing the transient syncing_files "try again later" failure. Deploy on push to main (and manual dispatch) only; the Go Run Tests suite is unrelated to the Hugo site so it should not gate it. Co-authored-by: Cursor <cursoragent@cursor.com>
Reusable subscribe partial (single source of truth) rendered in a footer
override above the copyright, plus a {{< subscribe >}} shortcode wrapper for
in-content use. Labels are i18n-driven with EN/LT strings; styles live in
custom.css. The field has no id so the form stays valid if it appears twice
on a page (footer + shortcode).
Co-authored-by: Cursor <cursoragent@cursor.com>
… + sidebar Replace the always-on footer form with a subscribe modal (native <dialog>) rendered once per page via the body-end hook. Add an email icon among the follow icons (header) and a "Get new posts by email" button under the sidebar intro; both open the same modal. Modal reuses the subscribe partial, is centered over a dimmed backdrop, and closes via the button, backdrop, or Esc. Adds a "close" i18n string (EN/LT) and modal/trigger styles. Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
rsssocial entry in the EN/LT menus so the feed icon appears alongside the other social links and points to each language's home feed. Autodiscovery + per-section feeds were already emitted by the theme.xml-stylesheetPI to the feed and shipstatic/feed.xsl, so browsers render a styled page (title, posts, subscribe hint) instead of raw XML, while feed readers keep consuming the underlying RSS.mainexactly once per push and drop the duplicateworkflow_runtrigger. Merging previously fired two competing Pages deployments (push + Run Tests hand-off) that raced in thepagesconcurrency group and caused the transientsyncing_files → "try again later"failure.Notes
Test plan
index.xml/lt/index.xml.