Merge pull request #14430 from jchyb/scaladoc/fix-i14421

Scaladoc: Make member filtering case-insensitive
scala · Feb 15, 2022 · 990a367 · 990a367
2 parents 4a1586c + 025aa18
commit 990a367
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/scaladoc/resources/dotty_res/scripts/components/DocumentableList.js b/scaladoc/resources/dotty_res/scripts/components/DocumentableList.js
@@ -139,7 +139,9 @@ class List {
       : includesInputValue()
 
     function includesInputValue() {
-      return elementData.name.includes(filter.value) || elementData.description.includes(filter.value);
+      const lcValue = filter.value.toLowerCase()
+      return elementData.name.toLowerCase().includes(lcValue) 
+          || elementData.description.toLowerCase().includes(lcValue);
     }
 
     function areFiltersFromElementSelected() {

diff --git a/scaladoc/resources/dotty_res/scripts/components/Filter.js b/scaladoc/resources/dotty_res/scripts/components/Filter.js
@@ -75,12 +75,14 @@ class Filter {
   * @returns { Filters }
   */
   _generateFiltersOnTyping(value) {
+    const lcValue = value.toLowerCase()
+
     const elementsDatasets = this.elementsRefs
       .filter(element => {
-        const name = getElementTextContent(getElementNameRef(element));
-        const description = getElementTextContent(getElementDescription(element));
+        const lcName = getElementTextContent(getElementNameRef(element)).toLowerCase();
+        const lcDescription = getElementTextContent(getElementDescription(element)).toLowerCase();
 
-        return name.includes(value) || description.includes(value);
+        return lcName.includes(lcValue) || lcDescription.includes(lcValue);
       })
       .map(element => this._getDatasetWithKeywordData(element.dataset))