@internetarchive/collection-browser 0.3.2-alpha.4 → 0.3.2

package/src/tiles/image-block.ts

@@ -22,9 +22,8 @@ export class ImageBlock extends LitElement {
   @property({ type: String }) viewSize: string = 'desktop';
 
   render() {
-    if (!this.model?.identifier) {
-      return nothing;
-    }
+    if (!this.model?.identifier) return nothing;
+
     return html`
       <div class=${classMap(this.baseClass)}>
         <item-image
@@ -52,9 +51,9 @@ export class ImageBlock extends LitElement {
   private get iconOverlayTemplate() {
     if (!this.isListTile) return nothing;
 
-    if (!this.model?.loginRequired && !this.model?.contentWarning) {
+    if (!this.model?.loginRequired && !this.model?.contentWarning)
       return nothing;
-    }
+
     return html`
       <icon-overlay
         .loggedIn=${this.loggedIn}
@@ -65,13 +64,11 @@ export class ImageBlock extends LitElement {
   }
 
   private get textOverlayTemplate() {
-    if (this.isListTile) {
-      return nothing;
-    }
+    if (this.isListTile) return nothing;
 
-    if (!this.model?.loginRequired && !this.model?.contentWarning) {
+    if (!this.model?.loginRequired && !this.model?.contentWarning)
       return nothing;
-    }
+
     return html`
       <text-overlay
         .loggedIn=${this.loggedIn}

package/src/tiles/list/tile-list-compact.ts

@@ -38,6 +38,7 @@ export class TileListCompact extends LitElement {
           .isCompactTile=${true}
           .isListTile=${true}
           .viewSize=${this.classSize}
+          .loggedIn=${this.loggedIn}
         >
         </image-block>
         <div id="title">${DOMPurify.sanitize(this.model?.title ?? '')}</div>

package/src/tiles/list/tile-list.ts

@@ -125,6 +125,7 @@ export class TileList extends LitElement {
         .isCompactTile=${false}
         .isListTile=${true}
         .viewSize=${this.classSize}
+        .loggedIn=${this.loggedIn}
       >
       </image-block>
     `;

package/src/tiles/overlay/icon-overlay.ts

@@ -11,10 +11,9 @@ export class IconOverlay extends LitElement {
   @property({ type: Boolean }) loginRequired = false;
 
   render() {
-    if (this.loginRequired && !this.loggedIn) {
-      return html`${loginRequiredIcon} `;
-    }
-    return html`${restrictedIcon}`;
+    return this.loginRequired && !this.loggedIn
+      ? html`${loginRequiredIcon}`
+      : html`${restrictedIcon}`;
   }
 
   static get styles(): CSSResultGroup {

package/test/collection-browser.test.ts

@@ -227,6 +227,29 @@ describe('Collection Browser', () => {
     ).to.contains('Results');
   });
 
+  it('can search on demand if only search type has changed', async () => {
+    const searchService = new MockSearchService();
+
+    const el = await fixture<CollectionBrowser>(
+      html`<collection-browser
+        .searchService=${searchService}
+        .searchType=${SearchType.METADATA}
+      ></collection-browser>`
+    );
+
+    el.baseQuery = 'collection:foo';
+    await el.updateComplete;
+
+    el.searchType = SearchType.FULLTEXT;
+    await el.updateComplete;
+
+    // Haven't performed the search yet
+    expect(searchService.searchType).to.equal(SearchType.METADATA);
+
+    el.requestSearch();
+    expect(searchService.searchType).to.equal(SearchType.FULLTEXT);
+  });
+
   it('queries for collection names after a fetch', async () => {
     const searchService = new MockSearchService();
     const collectionNameCache = new MockCollectionNameCache();
@@ -239,7 +262,7 @@ describe('Collection Browser', () => {
       </collection-browser>`
     );
 
-    el.baseQuery = 'blahblah';
+    el.baseQuery = 'collection:foo';
     await el.updateComplete;
 
     expect(collectionNameCache.preloadIdentifiersRequested).to.deep.equal([