clean up some todos

Author: LFDanLu

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

@@ -51,7 +51,6 @@ export function useLoadMore(props: LoadMoreProps, ref: RefObject<HTMLElement | n
   let triggerLoadMore = useEffectEvent((entries: IntersectionObserverEntry[]) => {
     // Use "isIntersecting" over an equality check of 0 since it seems like there is cases where
     // a intersection ratio of 0 can be reported when isIntersecting is actually true
-    // TODO: firefox seems to gather multiple entries, will need to reproduce in a base repro
     for (let entry of entries) {
       if (entry.isIntersecting && !isLoading && !(collection && collectionAwaitingUpdate.current) && onLoadMore) {
         onLoadMore();

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

@@ -49,7 +49,6 @@ export interface ComboBoxRenderProps {
    */
   isRequired: boolean,
   // TODO: do we want loadingState for RAC Combobox or just S2
-  // TODO: move types somewhere common later
   /**
    * Whether the combobox is currently loading items.
    * @selector [data-loading]

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

@@ -528,7 +528,6 @@ export const UNSTABLE_GridListLoadingIndicator = createLeafComponent('loader', f
   return (
     <div
       role="row"
-      // TODO: do we need any of the other row props from useGridListItem? Rather not grab too much if this is all we need
       aria-rowindex={isVirtualized ? item.index + 1 : undefined}
       ref={ref}
       {...mergeProps(filterDOMProps(props as any))}

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

@@ -248,9 +248,8 @@ function ListBoxInner<T extends object>({state: inputState, props, listBoxRef}:
   // dropdown opens but the current state gives the user more freedom as to whether they would like to pre-fetch or not
   useLoadMore(memoedLoadMoreProps, listBoxRef);
 
-  // TODO: add loading indicator to ListBox so user can render that when loading. Think about if completely empty state
-  // do we leave it up to the user to setup the two states for empty and empty + loading? Do we add a data attibute/prop/renderprop to ListBox
-  // for isLoading
+  // TODO: Think about if completely empty state. Do we leave it up to the user to setup the two states for empty and empty + loading?
+  // Do we add a data attibute/prop/renderprop to ListBox for isLoading
   return (
     <FocusScope>
       <div

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

@@ -60,7 +60,6 @@ export interface SelectRenderProps {
    * @selector [data-required]
    */
   isRequired: boolean,
-  // TODO: move types somewhere common later
   /**
    * Whether the select is currently loading items.
    * @selector [data-loading]