Skip to content

Commit acbfbb8

Browse files
mozziusclaude
andcommitted
fix following pill comments
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent 54a4348 commit acbfbb8

3 files changed

Lines changed: 8 additions & 12 deletions

File tree

‎src/features/followingV2/home/PostFeed.tsx‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -484,9 +484,9 @@ let PostFeed = ({
484484
/*
485485
* Anchored Following's checks, made by the view on screen only. Each one
486486
* that finds newer posts stages them above the reader at rest (see
487-
* `usePostFeedPrepend`) and lights the Home dot. Only one on a significant
488-
* return to the app, or one that joins a restore's prepend, offers them with
489-
* the pill. With no boundary to fetch above, as for an empty feed, it peeks
487+
* `usePostFeedPrepend`) and lights the Home dot. A significant return or a
488+
* restore puts the pill up for them, and while it's up every check's find
489+
* joins it. With no boundary to fetch above, as for an empty feed, it peeks
490490
* for the Home dot instead, as other feeds do.
491491
*/
492492
useNewPostsCheck({

‎src/features/followingV2/home/queries/postFeed.ts‎

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -628,9 +628,9 @@ function isSameFeedItem(
628628
* While a page is staged, the next prepend fetches from the same `since` as it
629629
* did and replaces it, so however long the reader stays put, what's above them
630630
* is one page of at most {@link PREPEND_LIMIT} posts. Once it's read, the next
631-
* one goes above it. `staged` is the page for the pill, which goes up if a
632-
* restore or a significant return to the app put it there, or `check` has
633-
* offered it since, and takes in every page staged while it's up.
631+
* one goes above it. The pill goes up for a `staged` page that a restore or a
632+
* significant return put there, or that `check` has offered since, and takes
633+
* in every page staged while it's up (see `usePrependPill`).
634634
*
635635
* A page only ever replaces the staged one if it has all of its posts, as
636636
* the list holds the reader's place only for posts added above the others:
@@ -796,10 +796,7 @@ export function usePostFeedPrepend(
796796
}
797797
setStaged({
798798
fetchedAt: page.fetchedAt,
799-
/*
800-
* Only if this prepend offers it: the pill keeps a page that replaces
801-
* one it has, while it's up.
802-
*/
799+
// A replacement isn't offered. The pill takes it in itself if it's up.
803800
offered: isOffering.current,
804801
isFull,
805802
})

‎src/features/followingV2/home/useSettleScrollHandlers.ts‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,7 @@ import {SETTLE_AT_TOP_LIMIT, type SettleAtTopHandlers} from './useSettleAtTop'
1212

1313
/**
1414
* What the list reports on the JS thread: what {@link SettleAtTopHandlers}
15-
* need, and when it reaches its top, which reads the posts staged above the
16-
* reader (see `usePostFeedPrepend`) and takes the pill down.
15+
* need, and when it reaches its top.
1716
*/
1817
export type ListScrollReports = SettleAtTopHandlers & {
1918
/** A scroll event found the list at the top where the last one didn't. */

0 commit comments

Comments
 (0)