Skip to content

Re-enable news summaries and improve reader page formatting#340

Merged
asim merged 1 commit intomainfrom
claude/track-api-usage-L6x8s
Mar 22, 2026
Merged

Re-enable news summaries and improve reader page formatting#340
asim merged 1 commit intomainfrom
claude/track-api-usage-L6x8s

Conversation

@asim
Copy link
Member

@asim asim commented Mar 22, 2026

  1. News summaries: Remove the early return that was disabling AI summary generation. Summaries were useful and worth the cost.

  2. Reader formatting: Replace plain-text extraction with HTML sanitization. Instead of stripping all HTML tags (which caused words to merge and lost all structure), the reader now preserves safe structural HTML — headings, paragraphs, lists, blockquotes, emphasis, links, tables, and code blocks. Dangerous elements (scripts, styles, iframes, nav, ads) are still removed.

  3. Reader CSS: Add dedicated .reader-content styles for clean typography — proper spacing for paragraphs, headings, lists, blockquotes, code blocks, and tables.

https://claude.ai/code/session_016UhaM3HefwZjArvB7hHbpE

1. News summaries: Remove the early return that was disabling AI summary
   generation. Summaries were useful and worth the cost.

2. Reader formatting: Replace plain-text extraction with HTML sanitization.
   Instead of stripping all HTML tags (which caused words to merge and lost
   all structure), the reader now preserves safe structural HTML — headings,
   paragraphs, lists, blockquotes, emphasis, links, tables, and code blocks.
   Dangerous elements (scripts, styles, iframes, nav, ads) are still removed.

3. Reader CSS: Add dedicated .reader-content styles for clean typography —
   proper spacing for paragraphs, headings, lists, blockquotes, code blocks,
   and tables.

https://claude.ai/code/session_016UhaM3HefwZjArvB7hHbpE
@asim asim merged commit 4e6bb1a into main Mar 22, 2026
1 of 2 checks 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