Skip to content

fix(home): defer unopened post carousels - #654

Merged
harlan-zw merged 1 commit into
mainfrom
fix/home-hidden-content
Oct 14, 2026
Merged

harlan-zw merged 1 commit into
mainfrom
fix/home-hidden-content

Conversation

@harlan-zw

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

Follow-up to #653.

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

Closed homepage post panels add every carousel entry to initial HTML, repeating the visible source quote. The homepage now mounts each carousel when first opened and preserves its scroll position when folded. Full trending boards still include every source post in initial HTML.

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

@harlan-zw

Copy link
Copy Markdown
Collaborator Author

This comment was written by the submitting Agent.

The local preview used captured public feed responses. I switched the homepage deferral flag off, then on. With identical responses, afdocs's HTML converter measured 61,481 → 47,866 characters, a 22% reduction. The lead source quotes stayed rendered. The afdocs 0.22.2 rendering and HTML size checks both passed. This result covers one page and two checks.

Browser checks passed at 375px, 768px, and 1280px. Opening showed all six supplied posts. Keyboard Space folded and reopened the panel. The carousel retained its 312px scroll position, and the page had no horizontal overflow.

Temporary regressions covered deferred entries, mounted carousel state, and complete default board serialization. The original component failed the deferral case; the repaired component passed all three. Temporary tests and HTTP fixtures were removed before the final build and commit.

@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 14, 2026
@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. harlan-agent-running An Agent holds a Task on this issue or pull request right now. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. labels Oct 14, 2026
@harlan-github-agent

harlan-github-agent Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

🤖 MERGED

Harlan Agent Kit posted this automated review. It is not Harlan's personal review or approval. AI open source policy. Last updated: 2026-10-14 03:45 UTC.

GitHub merged this pull request.

No material findings were recorded.

61d1d7d8-ae33-4e86-92c3-1912a29d5aa8

@harlan-github-agent harlan-github-agent Bot added harlan-agent-pending The automated Review is waiting on a gate for this head commit. and removed harlan-agent-running An Agent holds a Task on this issue or pull request right now. harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. labels Oct 14, 2026

@harlan-github-agent harlan-github-agent Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Harlan Agent Kit posted this automated review. AI open source policy.

Automated Review: READY.

Read the Review results

@harlan-github-agent harlan-github-agent Bot added harlan-agent-ready The automated Review passed every gate on this head commit. and removed harlan-agent-pending The automated Review is waiting on a gate for this head commit. labels Oct 14, 2026
@harlan-zw
harlan-zw merged commit 2e2a1e5 into main Oct 14, 2026
8 checks passed
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-ready The automated Review passed every gate on this head commit. label Oct 14, 2026
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.

1 participant