📰 News Pop-Up Content Missing or Inconsistent Across Sources

[muk2]

When selecting a news headline and pressing Enter, the pop-up window opens but article content often does not render.

Observed Behavior
• Hacker News articles:
• Pop-up opens
• Content does not load
• Ars Technica:
• Partial or inconsistent content rendering

Expected Behavior
• Pop-up window displays article content reliably
• Graceful fallback when full content cannot be fetched

Possible Causes
• HTML parsing differences across sources
• Missing fetch / render step after pop-up creation
• Source-specific scraping logic not standardized

Suggested Improvements
• Normalize article fetching per source
• Add loading / error states
• Show summary or external link if full content unavailable

[muk2]

@claude PR number #21 was incorrectly implemented as enter key was not wanted to open article. Wanted to keep the pop up that appears when enter is clicked on a news article. Currently pressing 'o' opens article once in pop up which is expected behavior. the pop up terminal is missing the article text and comments if there are any. please open a New PR off of main addressing this issue

[claude]

Claude finished @muk2's task in 3m 12s —— View job • claude/issue-17-20260201-1818 • Create PR ➔

---

Working on Issue #17: News Pop-Up Content Missing

Tasks

• Read CLAUDE.md and understand repo structure
• Examine PR Add Enter key to open selected items in browser #21 to understand incorrect changes
• Locate news pop-up implementation code
• Identify article content fetching logic
• Fetch and display HN comments in pop-up
• Update article reader to show comments section
• Commit and push changes
• Provide PR creation link

Implementation Summary

I've implemented a solution that fetches and displays Hacker News comments in the article pop-up:

Changes made:

1. Extended HnStory struct (feeds/mod.rs:29-43):

   • Added text field for story text (Ask HN, Show HN posts)
   • Added comments field with top 5 comments preview
   • Added HnComment struct to hold comment data

2. Updated HN API fetcher (feeds/hackernews.rs):

   • Fetches top 5 comments for each story
   • Added fetch_comment() method to retrieve comment data
   • Comments are fetched during initial story load

3. Enhanced article reader display (ui/widgets/hackernews.rs:162-206):

   • Displays story text if available (for Ask HN/Show HN posts)
   • Shows discussion preview with top comments
   • Indicates how many more comments are available
   • Maintains "press 'o' to view all" call-to-action

Result:

• ✅ Enter key still opens the pop-up (unchanged behavior)
• ✅ Pop-up now displays meaningful content instead of "No description available"
• ✅ 'o' key still opens full article/discussion in browser (unchanged behavior)
• ✅ Fast performance by limiting to 5 comments per story

Next Steps

--- | Branch