Add lazy loading of results. Improve UX.

Author: pikinier20

scala3doc-js/resources/scala3doc-searchbar.css

@@ -1,7 +1,3 @@
-:root {
-    --inkuire-logo-size: 20px;
-}
-
 /* button */
 .search span {
     background: #ED3522;

scala3doc-js/src/searchbar/SearchbarComponent.scala

@@ -6,7 +6,7 @@ import org.scalajs.dom.html.Input
 class SearchbarComponent(val callback: (String) => List[PageEntry]):
   extension (p: PageEntry)
     def toHTML =
-      val wrapper = document.createElement("div")
+      val wrapper = document.createElement("div").asInstanceOf[html.Div]
       wrapper.classList.add("scala3doc-searchbar-result")
       wrapper.classList.add("monospace")
 
@@ -25,8 +25,23 @@ class SearchbarComponent(val callback: (String) => List[PageEntry]):
 
   def handleNewQuery(query: String) =
     val result = callback(query).map(_.toHTML)
+    resultsDiv.scrollTop = 0
     while (resultsDiv.hasChildNodes()) resultsDiv.removeChild(resultsDiv.lastChild)
-    result.foreach(resultsDiv.appendChild)
+    val fragment = document.createDocumentFragment()
+    result.take(100).foreach(fragment.appendChild)
+    resultsDiv.appendChild(fragment)
+    def loadMoreResults(result: List[raw.HTMLElement]): Unit = {
+      resultsDiv.onscroll = (event: Event) => {
+          if (resultsDiv.scrollHeight - resultsDiv.scrollTop == resultsDiv.clientHeight)
+          {
+              val fragment = document.createDocumentFragment()
+              result.take(100).foreach(fragment.appendChild)
+              resultsDiv.appendChild(fragment)
+              loadMoreResults(result.drop(100))
+          }
+      }
+    }
+    loadMoreResults(result.drop(100))
 
   private val logoClick: html.Div =
     val span = document.createElement("span").asInstanceOf[html.Span]
@@ -39,7 +54,7 @@ class SearchbarComponent(val callback: (String) => List[PageEntry]):
       else document.body.appendChild(rootDiv)
 
     val element = createNestingDiv("search-content")(
-      createNestingDiv("search-conatiner")(
+      createNestingDiv("search-container")(
         createNestingDiv("search")(
           span
         )
@@ -71,9 +86,9 @@ class SearchbarComponent(val callback: (String) => List[PageEntry]):
 
   private val rootDiv: html.Div =
     val element = document.createElement("div").asInstanceOf[html.Div]
-    element.addEventListener("click", (e: Event) => e.stopPropagation())
-    logoClick.addEventListener("click", (e: Event) => e.stopPropagation())
-    document.body.addEventListener("click", (e: Event) =>
+    element.addEventListener("mousedown", (e: Event) => e.stopPropagation())
+    logoClick.addEventListener("mousedown", (e: Event) => e.stopPropagation())
+    document.body.addEventListener("mousedown", (e: Event) =>
       if (document.body.contains(element)) {
         document.body.removeChild(element)
       }

scala3doc-js/src/searchbar/engine/SearchbarEngine.scala

@@ -4,7 +4,6 @@ import math.Ordering.Implicits.seqOrdering
 
 class SearchbarEngine(pages: List[PageEntry]) {
   def query(query: List[Matchers]): List[PageEntry] = {
-    // Results list is limited to 100 entries for performance reasons
-    pages.map(p => p -> query.map(_(p))).filterNot(_(1).exists(_ < 0)).sortBy(_(1)).map(_(0)).take(100)
+    pages.map(p => p -> query.map(_(p))).filterNot(_(1).exists(_ < 0)).sortBy(_(1)).map(_(0))
   }
 }

scala3doc/src/dotty/renderers/ScalaSearchbarDataInstaller.scala

@@ -61,7 +61,7 @@ class ScalaSearchbarDataInstaller(val ctx: DokkaContext) extends SearchbarDataIn
   }
 
   private def createAcronym(s: String): Option[String] =
-    if s.head.isUpper then Some(s.filter(_.isUpper)) else None
+    s.headOption.filter(_.isUpper).map(_ => s.filter(_.isUpper))
 
   override def generatePagesList(): String = {
     val mapper = jacksonObjectMapper()