fix: app hang on attach contact sometimes

Closes #5723.

It seems that sometimes this causes front-end to basically
infinitely spam the backend with `get_contacts_by_ids`.

`isContactLoaded` also starts returning `true`
as soon as we have started loading a contact,
and not only when we have loaded it.

This might not completely fix such a potential bug,
but this should at least make it not happen in practice.

This bug has likely been surfaced by
9bb605c
(#5581).

Author: WofWca

packages/frontend/src/components/dialogs/SelectContact/index.tsx

@@ -28,10 +28,8 @@ export default function SelectContactDialog({
   onOk: (contact: T.Contact) => void
 } & DialogProps) {
   const [queryStr, setQueryStr] = useState('')
-  const { contactIds, contactCache, loadContacts } = useLazyLoadedContacts(
-    C.DC_GCL_ADD_SELF,
-    queryStr
-  )
+  const { contactIds, contactCache, loadContacts, isContactLoaded } =
+    useLazyLoadedContacts(C.DC_GCL_ADD_SELF, queryStr)
   const tx = useTranslationFunction()
 
   const selectContactListRef = useRef<HTMLDivElement>(null)
@@ -71,12 +69,7 @@ export default function SelectContactDialog({
                 ref={infiniteLoaderRef}
                 itemCount={contactIds.length}
                 loadMoreItems={loadContacts}
-                // perf: consider using `isContactLoaded` from `useLazyLoadedContacts`
-                // otherwise sometimes we might load the same contact twice (performance thing)
-                // See https://github.com/bvaughn/react-window/issues/765
-                isItemLoaded={index =>
-                  contactCache[contactIds[index]] != undefined
-                }
+                isItemLoaded={isContactLoaded}
                 // minimumBatchSize={100}
               >
                 {({ onItemsRendered, ref }) => (