fix: include more search results in linking autocomplete menu

packages/web/src/javascripts/Components/ContentListView/Daily/DailyItemCell.tsx

@@ -9,7 +9,6 @@ import { ListableContentItem } from '../Types/ListableContentItem'
 import { DailyItemsDay } from './DailyItemsDaySection'
 import { ListItemTitle } from '../ListItemTitle'
 import { EmptyPlaceholderBars } from './EmptyPlaceholderBars'
-import { isMobileScreen } from '@/Utils'
 
 type DaySquareProps = {
   day: number
@@ -73,7 +72,7 @@ export const DailyItemCell = forwardRef(
               {!item && (
                 <div className="w-full">
                   <div className="break-word mr-2 font-semibold">{formatDateAndTimeForNote(section.date, false)}</div>
-                  <EmptyPlaceholderBars rows={isMobileScreen() ? 2 : 4} />
+                  <EmptyPlaceholderBars rows={1} />
                 </div>
               )}
             </div>

packages/web/src/javascripts/Components/NoteView/NoteView.tsx

@@ -95,7 +95,7 @@ class NoteView extends AbstractComponent<NoteViewProps, State> {
   private protectionTimeoutId: ReturnType<typeof setTimeout> | null = null
   private noteViewElementRef: RefObject<HTMLDivElement>
   private editorContentRef: RefObject<HTMLDivElement>
-  private plainEditorRef?: RefObject<PlainEditorInterface>
+  private plainEditorRef?: PlainEditorInterface
 
   constructor(props: NoteViewProps) {
     super(props, props.application)
@@ -231,6 +231,18 @@ class NoteView extends AbstractComponent<NoteViewProps, State> {
     }
   }
 
+  setPlainEditorRef = (ref: PlainEditorInterface | undefined) => {
+    this.plainEditorRef = ref || undefined
+
+    if (!this.plainEditorRef) {
+      return
+    }
+
+    if (this.controller.isTemplateNote && this.controller.templateNoteOptions?.autofocusBehavior === 'editor') {
+      this.plainEditorRef.focus()
+    }
+  }
+
   override componentDidUpdate(_prevProps: NoteViewProps, prevState: State): void {
     if (
       this.state.showProtectedWarning != undefined &&
@@ -569,7 +581,7 @@ class NoteView extends AbstractComponent<NoteViewProps, State> {
     }
 
     currentTarget.blur()
-    this.plainEditorRef?.current?.focus()
+    this.plainEditorRef?.focus()
   }
 
   onTitleChange: ChangeEventHandler<HTMLInputElement> = ({ currentTarget }) => {
@@ -944,7 +956,7 @@ class NoteView extends AbstractComponent<NoteViewProps, State> {
             <PlainEditor
               application={this.application}
               spellcheck={this.state.spellcheck}
-              ref={this.plainEditorRef}
+              ref={(ref) => this.setPlainEditorRef(ref || undefined)}
               controller={this.controller}
               locked={this.state.noteLocked}
               onFocus={this.onPlainFocus}

packages/web/src/javascripts/Components/NoteView/PlainEditor/PlainEditor.tsx

@@ -33,7 +33,7 @@ export type PlainEditorInterface = {
 }
 
 export const PlainEditor = forwardRef<PlainEditorInterface, Props>(
-  ({ application, spellcheck, controller, locked, onFocus, onBlur }: Props, ref) => {
+  ({ application, spellcheck, controller, locked, onFocus, onBlur }, ref) => {
     const [editorText, setEditorText] = useState<string | undefined>()
     const [textareaUnloading, setTextareaUnloading] = useState(false)
     const [lineHeight, setLineHeight] = useState<EditorLineHeight | undefined>()
@@ -239,7 +239,7 @@ export const PlainEditor = forwardRef<PlainEditorInterface, Props>(
         onFocus={onContentFocus}
         onBlur={onContentBlur}
         readOnly={locked}
-        ref={(ref) => ref && onRef(ref)}
+        ref={onRef}
         spellCheck={spellcheck}
         value={editorText}
         className={classNames(

packages/web/src/javascripts/Controllers/ItemList/ItemListController.ts

@@ -610,7 +610,7 @@ export class ItemListController extends AbstractViewController implements Intern
   async createNewNoteController(
     title?: string,
     createdAt?: Date,
-    autofocusBehavior?: TemplateNoteViewAutofocusBehavior,
+    autofocusBehavior: TemplateNoteViewAutofocusBehavior = 'editor',
   ) {
     const selectedTag = this.navigationController.selected
 

packages/web/src/javascripts/Utils/Items/Search/getSearchResults.ts

@@ -6,8 +6,12 @@ import { isSearchResultExistingTag } from './isSearchResultExistingTag'
 import { ItemLink } from './ItemLink'
 import { LinkableItem } from './LinkableItem'
 
-const ResultLimitPerContentType = 5
+const MaxLinkedResults = 50
-const MaxLinkedResults = 20
+
+function resultLimitForSearchQuery(query: string): number {
+  const limitPerContentType = 10
+  return Math.max(limitPerContentType, query.length * 3)
+}
 
 export function getLinkingSearchResults(
   searchQuery: string,
@@ -77,27 +81,22 @@ export function getLinkingSearchResults(
 
     const enforceResultLimit = options.contentType == null
 
+    const limitPerContentType = resultLimitForSearchQuery(searchQuery)
+
     if (
       item.content_type === ContentType.Tag &&
-      (!enforceResultLimit ||
+      (!enforceResultLimit || (unlinkedTags.length < limitPerContentType && item.content_type === ContentType.Tag))
-        (unlinkedTags.length < ResultLimitPerContentType && item.content_type === ContentType.Tag))
     ) {
       unlinkedTags.push(item)
       continue
     }
 
-    if (
+    if (item.content_type === ContentType.Note && (!enforceResultLimit || unlinkedNotes.length < limitPerContentType)) {
-      item.content_type === ContentType.Note &&
-      (!enforceResultLimit || unlinkedNotes.length < ResultLimitPerContentType)
-    ) {
       unlinkedNotes.push(item)
       continue
     }
 
-    if (
+    if (item.content_type === ContentType.File && (!enforceResultLimit || unlinkedFiles.length < limitPerContentType)) {
-      item.content_type === ContentType.File &&
-      (!enforceResultLimit || unlinkedFiles.length < ResultLimitPerContentType)
-    ) {
       unlinkedFiles.push(item)
       continue
     }