Batch 1: Feed experience — load more, session cache, safe share button#27
Open
tinkerwise wants to merge 6 commits into
Open
Batch 1: Feed experience — load more, session cache, safe share button#27tinkerwise wants to merge 6 commits into
tinkerwise wants to merge 6 commits into
Conversation
Feed experience (Batch 1): - MAX_VISIBLE_ARTICLES bumped to 30; load-more adds 20 at a time - loadMoreVisible state resets on filter/sort/search/category changes - Share button converted from inline onclick to delegated data-action listener Housekeeping (Batch 4): - Audio path in easter-eggs.js uses BASE constant instead of hardcoded URL - Auto-refresh skips fetches between midnight and 8am local time - City Connect Friday auto-theme only applies when user has no saved preference - Reader modal converted from hardcoded light colors to CSS theme variables API abstraction (Batch 5 partial): - New mlbApi.js with named functions for every MLB Stats API endpoint https://claude.ai/code/session_01C1vhzbYUCRYNikqatrcAYN
- Show 30 articles initially (up from 10), Load More reveals 20 at a time - sessionStorage cache (key: yardreport_feed_cache, TTL 20 min) serves stale-while-revalidate: instant render on reload, silent background refresh - Share button inline onclick replaced with event-delegated data-action listener - Load More button styled with orange accent (#df4601) consistent with dark theme - loadMoreVisible resets to 0 on search/sort/filter/category changes https://claude.ai/code/session_01C1vhzbYUCRYNikqatrcAYN
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.
Closes #22
Summary
sessionStoragefeed cache (20-min TTL) so return visits render immediately instead of showing a blank loading stateonclickshare button with a delegated event listener usingdata-action="share"Files changed
src/scripts/feedDisplay.js—MAX_VISIBLE_ARTICLES→ 30,LOAD_MORE_BATCH = 20src/scripts/feeds.js— cache read/write, load more wiring, delegated share listenersrc/scripts/state.js—loadMoreVisiblecountersrc/scripts/app.js— resetloadMoreVisibleon filter changespublic/style.css—.load-more-btnstylesTest plan
onclickin sourceGenerated by Claude Code