get rid of extra todos and obsolte test

Author: LFDanLu

packages/@react-aria/utils/src/useLoadMoreSentinel.ts

@@ -26,7 +26,6 @@ export interface LoadMoreSentinelProps extends Omit<AsyncLoadable, 'isLoading'>
    * @default 1
    */
   scrollOffset?: number
-  // TODO: Maybe include a scrollRef option so the user can provide the scrollParent to compare against instead of having us look it up
 }
 
 export function UNSTABLE_useLoadMoreSentinel(props: LoadMoreSentinelProps, ref: RefObject<HTMLElement | null>): void {

packages/@react-spectrum/s2/chromatic/Combobox.stories.tsx

@@ -83,7 +83,6 @@ export const WithEmptyState = {
   }
 };
 
-// TODO: this one is probably not great for chromatic since it has the spinner, check if ignoreSelectors works for it
 export const WithInitialLoading = {
   ...EmptyCombobox,
   args: {

packages/react-aria-components/src/GridList.tsx

@@ -532,7 +532,7 @@ export const UNSTABLE_GridListLoadingSentinel = createLeafComponent('loader', fu
 
   return (
     <>
-      {/* TODO: Alway render the sentinel. Will need to figure out how best to position this in cases where the user is using flex + gap (this would introduce a gap even though it doesn't take room) */}
+      {/* Alway render the sentinel. For now onus is on the user for styling when using flex + gap (this would introduce a gap even though it doesn't take room) */}
       {/* @ts-ignore - compatibility with React < 19 */}
       <div style={{position: 'relative', width: 0, height: 0}} inert={inertValue(true)} >
         <div data-testid="loadMoreSentinel" ref={sentinelRef} style={{position: 'absolute', height: 1, width: 1}} />

packages/react-aria-components/src/ListBox.tsx

@@ -509,7 +509,7 @@ export const UNSTABLE_ListBoxLoadingSentinel = createLeafComponent('loader', fun
 
   return (
     <>
-      {/* TODO: Alway render the sentinel. Will need to figure out how best to position this in cases where the user is using flex + gap (this would introduce a gap even though it doesn't take room) */}
+      {/* Alway render the sentinel. For now onus is on the user for styling when using flex + gap (this would introduce a gap even though it doesn't take room) */}
       {/* @ts-ignore - compatibility with React < 19 */}
       <div style={{position: 'relative', width: 0, height: 0}} inert={inertValue(true)} >
         <div data-testid="loadMoreSentinel" ref={sentinelRef} style={{position: 'absolute', height: 1, width: 1}} />

packages/react-aria-components/src/Table.tsx

@@ -1397,8 +1397,7 @@ export const UNSTABLE_TableLoadingSentinel = createLeafComponent('loader', funct
 
   return (
     <>
-      {/* TODO weird structure? Renders a extra row but we hide via inert so maybe ok? */}
-      {/* TODO: Will need to figure out how best to position this in cases where the user is using flex + gap (this would introduce a gap even though it doesn't take room) */}
+      {/* Alway render the sentinel. For now onus is on the user for styling when using flex + gap (this would introduce a gap even though it doesn't take room) */}
       {/* @ts-ignore - compatibility with React < 19 */}
       <TR style={{position: 'relative', width: 0, height: 0}} inert={inertValue(true)} >
         <TD data-testid="loadMoreSentinel" ref={sentinelRef} style={{position: 'absolute', height: 1, width: 1}} />

packages/react-aria-components/stories/Table.stories.tsx

@@ -844,7 +844,6 @@ function VirtualizedTableWithEmptyState(args) {
             <MyColumn>Baz</MyColumn>
           </TableHeader>
           <MyTableBody
-            // TODO: adjust this as well
             isLoading={args.isLoading}
             renderEmptyState={() => renderEmptyLoader({isLoading: !args.showRows && args.isLoading})}
             rows={!args.showRows ? [] : rows}>

packages/react-aria-components/test/Table.test.js

@@ -2003,23 +2003,6 @@ describe('Table', () => {
       expect(onLoadMore).toHaveBeenCalledTimes(1);
     });
 
-    // TODO: decide if we want to allow customization for this (I assume we will)
-    it.skip('allows the user to customize the scrollOffset required to trigger onLoadMore', function () {
-      jest.spyOn(window.HTMLElement.prototype, 'scrollHeight', 'get').mockImplementation(() => 100);
-      jest.spyOn(window.HTMLElement.prototype, 'clientHeight', 'get').mockImplementation(() => 25);
-
-      let tree = render(<LoadMoreTable onLoadMore={onLoadMore} scrollOffset={2} />);
-
-      let scrollView = tree.getByTestId('scrollRegion');
-      expect(onLoadMore).toHaveBeenCalledTimes(0);
-
-      scrollView.scrollTop = 50;
-      fireEvent.scroll(scrollView);
-      act(() => {jest.runAllTimers();});
-
-      expect(onLoadMore).toHaveBeenCalledTimes(1);
-    });
-
     describe('virtualized', () => {
       let items = [];
       for (let i = 1; i <= 50; i++) {