Fun picker fixes

stylesheets/components/CompositionTextArea.scss

@@ -35,7 +35,7 @@
 
     &__input__scroller {
       max-height: 300px;
-      min-height: 300px;
+      min-height: 100px;
       padding: 16px;
       // Need more padding on the right to make room for floating emoji button
       padding-inline-end: 36px;

ts/components/CompositionTextArea.tsx

@@ -141,7 +141,7 @@ export function CompositionTextArea({
         isActive={isActive}
         isFormattingEnabled={isFormattingEnabled}
         inputApi={inputApiRef}
-        large
+        large={false}
         moduleClassName="CompositionTextArea__input"
         onEditorStateChange={handleChange}
         onPickEmoji={onPickEmoji}

ts/components/DraftGifMessageSendModal.tsx

@@ -84,7 +84,7 @@ export function DraftGifMessageSendModal(
           state={props.gifDownloadState.loadingState}
           width={props.gifSelection.width}
           height={props.gifSelection.height}
-          maxHeight={200}
+          maxHeight={256}
           aria-label={props.gifSelection.title}
           aria-describedby={descriptionId}
         />

ts/components/fun/base/FunImage.tsx

@@ -39,6 +39,8 @@ const FunImageBase = forwardRef(function FunImageBase(
       width={props.width}
       height={props.height}
       draggable={false}
+      decoding="async"
+      loading="lazy"
     />
   );
 });

ts/components/fun/data/emojis.ts

@@ -5,6 +5,7 @@ import { sortBy } from 'lodash';
 import { useEffect, useMemo, useState } from 'react';
 import { z } from 'zod';
 import * as log from '../../../logging/log';
+import * as Errors from '../../../types/errors';
 import type { LocaleEmojiListType } from '../../../types/emoji';
 import type { LocalizerType } from '../../../types/I18N';
 import { strictAssert } from '../../../util/assert';
@@ -569,7 +570,7 @@ function createEmojiSearchIndex(
 
 const FuseKeys: Array<Fuse.FuseOptionKey> = [
   { name: 'shortName', weight: 100 },
-  // { name: 'shortNames', weight: 1 },
+  { name: 'shortNames', weight: 1 },
   { name: 'emoticon', weight: 50 },
   { name: 'emoticons', weight: 1 },
 ];
@@ -648,7 +649,10 @@ export function useEmojiSearch(i18n: LocalizerType): EmojiSearch {
           setLocaleIndex(result);
         }
       } catch (error) {
-        log.error(`Failed to get localized emoji list for ${locale}`, error);
+        log.error(
+          `Failed to get localized emoji list for ${locale}`,
+          Errors.toLogFormat(error)
+        );
       }
     }
 

ts/state/ducks/composer.ts

@@ -402,6 +402,7 @@ export function saveDraftRecordingIfNeeded(): ThunkAction<
 type WithPreSendChecksOptions = Readonly<{
   message?: string;
   voiceNoteAttachment?: InMemoryAttachmentDraftType;
+  draftAttachments?: ReadonlyArray<AttachmentDraftType>;
 }>;
 
 async function withPreSendChecks(
@@ -425,6 +426,8 @@ async function withPreSendChecks(
   ]);
 
   const { message, voiceNoteAttachment } = options;
+  const draftAttachments =
+    options.draftAttachments ?? conversation.attributes.draftAttachments;
 
   try {
     dispatch(setComposerDisabledState(conversationId, true));
@@ -457,7 +460,7 @@ async function withPreSendChecks(
 
     if (
       !message?.length &&
-      !hasDraftAttachments(conversation.attributes.draftAttachments, {
+      !hasDraftAttachments(draftAttachments, {
         includePending: false,
       }) &&
       !voiceNoteAttachment