@internetarchive/collection-browser 1.10.1-alpha.4 → 1.11.0

package/dist/src/collection-browser.js

@@ -202,7 +202,6 @@ let CollectionBrowser = class CollectionBrowser extends LitElement {
      * in the options object.
      */
     clearFilters({ facets = true, dateRange = true, letterFilters = true, sort = false, } = {}) {
-        console.log('clearFilters', facets);
         // Don't bother clearing facets if none are checked, so that we don't
         // trigger unnecessary update cycles.
         if (facets && this.hasCheckedFacets) {
@@ -671,8 +670,6 @@ let CollectionBrowser = class CollectionBrowser extends LitElement {
     }
     updated(changed) {
         var _a;
-        console.log('--- UPDATED ---');
-        [...changed.entries()].forEach(([k, v]) => console.log(k, ':', v, '->', this[k]));
         if (changed.has('placeholderType') && this.placeholderType === null) {
             if (!this.leftColIntersectionObserver) {
                 this.setupLeftColumnScrollListeners();
@@ -688,20 +685,21 @@ let CollectionBrowser = class CollectionBrowser extends LitElement {
             changed.has('loggedIn')) {
             (_a = this.infiniteScroller) === null || _a === void 0 ? void 0 : _a.reload();
         }
-        if (changed.has('baseQuery') || changed.has('searchType')) {
+        if (changed.has('baseQuery') ||
+            changed.has('searchType') ||
+            changed.has('withinCollection')) {
             // Unless this query/search type update is from the initial page load or the
             // result of hitting the back button,
             // we need to clear any existing filters since they may no longer be valid for
             // the new set of search results.
             if (!this.historyPopOccurred && this.initialQueryChangeHappened) {
                 // Ordinarily, we leave the sort param unchanged between searches.
-                // However, if we are changing the target collection, we want the sort to be cleared too,
-                // since different collections can have different sorting options available.
+                // However, if we are changing the target collection itself, we want the sort cleared too,
+                // since different collections may have different sorting options available.
                 const shouldClearSort = changed.has('withinCollection') &&
                     !changed.has('selectedSort') &&
                     !changed.has('sortDirection');
                 // Otherwise, only clear filters that haven't been simultaneously applied in this update
-                console.log('clearing filters from update');
                 this.clearFilters({
                     sort: shouldClearSort,
                     facets: !changed.has('selectedFacets'),
@@ -739,16 +737,6 @@ let CollectionBrowser = class CollectionBrowser extends LitElement {
         if (changed.has('selectedCreatorFilter')) {
             this.sendFilterByCreatorAnalytics(changed.get('selectedCreatorFilter'));
         }
-        // if (changed.has('withinCollection')) {
-        //   // Because different collections can have different sort options available,
-        //   // we clear the applied sort when the target collection changes.
-        //   this.clearFilters({
-        //     sort: true,
-        //     facets: false,
-        //     dateRange: false,
-        //     letterFilters: false
-        //   });
-        // }
         if (changed.has('baseQuery') ||
             changed.has('searchType') ||
             changed.has('selectedTitleFilter') ||
@@ -904,7 +892,6 @@ let CollectionBrowser = class CollectionBrowser extends LitElement {
     }
     async handleQueryChange() {
         var _a;
-        console.log('handling query change');
         // only reset if the query has actually changed
         if (!this.searchService || this.pageFetchQueryKey === this.previousQueryKey)
             return;