@internetarchive/collection-browser 3.0.1 → 3.0.2

package/src/collection-facets.ts

@@ -397,6 +397,7 @@ export class CollectionFacets extends LitElement {
       ? html`<button
           class="expand-date-picker-btn"
           aria-hidden="true"
+          tabindex="-1"
           @click=${this.showDatePickerModal}
         >
           ${expandIcon}
@@ -425,7 +426,7 @@ export class CollectionFacets extends LitElement {
 
     return html`
       <histogram-date-range
-        class=${this.isTvSearch ? 'wide-inputs' : nothing}
+        class=${this.isTvSearch ? 'wide-inputs' : ''}
         .minDate=${minDate}
         .maxDate=${maxDate}
         .minSelectedDate=${this.minSelectedDate ?? minDate}
@@ -951,6 +952,7 @@ export class CollectionFacets extends LitElement {
           font-size: 1.2rem;
           text-decoration: none;
           padding: 0;
+          margin-top: 0.25rem;
           background: inherit;
           border: 0;
           color: var(--ia-theme-link-color, #4b64ff);

package/src/tiles/grid/collection-tile.ts

@@ -55,7 +55,7 @@ export class CollectionTile extends BaseTileComponent {
 
   private get getTitleTemplate() {
     return html`<div id="title">
-      <h4 class="truncated">${this.model?.title}</h4>
+      <h3 class="truncated">${this.model?.title}</h3>
     </div>`;
   }
 
@@ -123,7 +123,8 @@ export class CollectionTile extends BaseTileComponent {
           flex-grow: initial;
         }
 
-        h4.truncated {
+        h4.truncated,
+        h3.truncated {
           color: ${whiteColor};
         }
 

package/src/tiles/grid/item-tile.ts

@@ -60,9 +60,9 @@ export class ItemTile extends BaseTileComponent {
             ${this.imageBlockTemplate}
 
             <div id="title">
-              <h4 class="truncated" title=${ifDefined(itemTitle)}>
+              <h3 class="truncated" title=${ifDefined(itemTitle)}>
                 ${itemTitle}
-              </h4>
+              </h3>
             </div>
 
             ${this.volumeIssueTemplate}

package/src/tiles/grid/search-tile.ts

@@ -49,7 +49,7 @@ export class SearchTile extends BaseTileComponent {
 
   private get getTitleTemplate() {
     return html`<div id="title">
-      <h4 class="truncated">${this.model?.title}</h4>
+      <h3 class="truncated">${this.model?.title}</h3>
     </div>`;
   }
 
@@ -70,7 +70,8 @@ export class SearchTile extends BaseTileComponent {
           flex-grow: initial;
         }
 
-        h4.truncated {
+        h4.truncated,
+        h3.truncated {
           color: ${whiteColor};
           -webkit-line-clamp: 4;
         }

package/src/tiles/grid/styles/tile-grid-shared-styles.ts

@@ -72,7 +72,8 @@ export const baseTileStyles = css`
     -webkit-box-orient: vertical;
   }
 
-  h4.truncated {
+  h4.truncated,
+  h3.truncated {
     display: -webkit-box;
     margin: 0px;
     line-height: 15px;
@@ -114,7 +115,7 @@ export const baseTileStyles = css`
     align-items: center;
     background: rgba(220, 220, 220, 0.5);
     color: white;
-    font-size: 2rem;
+    font-size: 2.4rem;
     font-weight: bold;
     line-height: 1;
     text-shadow: black 1px 1px 3px;

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

@@ -4,7 +4,7 @@ import { join } from 'lit/directives/join.js';
 import { map } from 'lit/directives/map.js';
 import { unsafeHTML } from 'lit/directives/unsafe-html.js';
 import { customElement, property, state } from 'lit/decorators.js';
-import { msg } from '@lit/localize';
+import { msg, str } from '@lit/localize';
 import DOMPurify from 'dompurify';
 
 import type { SortParam } from '@internetarchive/search-service';
@@ -94,7 +94,11 @@ export class TileList extends BaseTileComponent {
       isCollection,
     );
 
-    return html`<a href=${href}>
+    return html`<a
+      id="image-link"
+      title=${msg(str`View ${this.model?.title}`)}
+      href=${href}
+    >
       <image-block
         .model=${this.model}
         .baseImageUrl=${this.baseImageUrl}
@@ -127,7 +131,11 @@ export class TileList extends BaseTileComponent {
   // Data templates
   private get iconRightTemplate() {
     return html`
-      <a id="icon-right" href=${this.mediatypeURL}>
+      <a
+        id="icon-right"
+        href=${this.mediatypeURL}
+        title=${msg(str`See more: ${this.model?.mediatype}`)}
+      >
         <tile-mediatype-icon .model=${this.model}> </tile-mediatype-icon>
       </a>
     `;

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

@@ -49,6 +49,20 @@ describe('List Tile', () => {
     expect(title?.getAttribute('href')).to.equal('/foo/bar');
   });
 
+  it('should add title to image link if provided', async () => {
+    const el = await fixture<TileList>(html`
+      <tile-list
+        .baseNavigationUrl=${''}
+        .model=${{ title: 'foo', href: '/foo/bar' }}
+      ></tile-list>
+    `);
+
+    const imageLink = el.shadowRoot?.querySelector('#image-link');
+
+    expect(imageLink).to.exist;
+    expect(imageLink?.getAttribute('title')).to.equal('View foo');
+  });
+
   it('should render with creator element but not dates', async () => {
     const el = await fixture<TileList>(html`
       <tile-list .model=${{ creators: ['someone'] }}></tile-list>
@@ -324,6 +338,7 @@ describe('List Tile', () => {
     expect(mediatypeLink?.getAttribute('href')).to.equal(
       `https://archive.org/details/texts`,
     );
+    expect(mediatypeLink?.getAttribute('title')).to.equal('See more: texts');
   });
 
   it('should render mediatype icon as link even with empty baseNavigationUrl', async () => {

package/tsconfig.json

@@ -1,20 +1,20 @@
-{
-  "compilerOptions": {
-    "target": "es2018",
-    "module": "esnext",
-    "moduleResolution": "node",
-    "noEmitOnError": true,
-    "lib": ["es2017", "dom", "dom.iterable"],
-    "strict": true,
-    "esModuleInterop": false,
-    "allowSyntheticDefaultImports": true,
-    "experimentalDecorators": true,
-    "importHelpers": true,
-    "outDir": "dist",
-    "sourceMap": true,
-    "inlineSources": true,
-    "rootDir": "./",
-    "declaration": true,
-  },
-  "include": ["src", "test", "index.ts", "types"],
-}
+{
+  "compilerOptions": {
+    "target": "es2018",
+    "module": "esnext",
+    "moduleResolution": "node",
+    "noEmitOnError": true,
+    "lib": ["es2017", "dom", "dom.iterable"],
+    "strict": true,
+    "esModuleInterop": false,
+    "allowSyntheticDefaultImports": true,
+    "experimentalDecorators": true,
+    "importHelpers": true,
+    "outDir": "dist",
+    "sourceMap": true,
+    "inlineSources": true,
+    "rootDir": "./",
+    "declaration": true,
+  },
+  "include": ["src", "test", "index.ts", "types"],
+}

package/web-dev-server.config.mjs

@@ -1,30 +1,30 @@
-// import { hmrPlugin, presets } from '@open-wc/dev-server-hmr';
-
-/** Use Hot Module replacement by adding --hmr to the start command */
-const hmr = process.argv.includes('--hmr');
-
-export default /** @type {import('@web/dev-server').DevServerConfig} */ ({
-  nodeResolve: true,
-  open: '/',
-  watch: !hmr,
-
-  /** Compile JS for older browsers. Requires @web/dev-server-esbuild plugin */
-  // esbuildTarget: 'auto'
-
-  /** Set appIndex to enable SPA routing */
-  // appIndex: 'demo/index.html',
-
-  /** Confgure bare import resolve plugin */
-  // nodeResolve: {
-  //   exportConditions: ['browser', 'development']
-  // },
-
-  plugins: [
-    /** Use Hot Module Replacement by uncommenting. Requires @open-wc/dev-server-hmr plugin */
-    // hmr && hmrPlugin({ exclude: ['**/*/node_modules/**/*'], presets: [presets.litElement] }),
-  ],
-
-  http2: true,
-  sslCert: './local.archive.org.cert',
-  sslKey: './local.archive.org.key',
-});
+// import { hmrPlugin, presets } from '@open-wc/dev-server-hmr';
+
+/** Use Hot Module replacement by adding --hmr to the start command */
+const hmr = process.argv.includes('--hmr');
+
+export default /** @type {import('@web/dev-server').DevServerConfig} */ ({
+  nodeResolve: true,
+  open: '/',
+  watch: !hmr,
+
+  /** Compile JS for older browsers. Requires @web/dev-server-esbuild plugin */
+  // esbuildTarget: 'auto'
+
+  /** Set appIndex to enable SPA routing */
+  // appIndex: 'demo/index.html',
+
+  /** Confgure bare import resolve plugin */
+  // nodeResolve: {
+  //   exportConditions: ['browser', 'development']
+  // },
+
+  plugins: [
+    /** Use Hot Module Replacement by uncommenting. Requires @open-wc/dev-server-hmr plugin */
+    // hmr && hmrPlugin({ exclude: ['**/*/node_modules/**/*'], presets: [presets.litElement] }),
+  ],
+
+  http2: true,
+  sslCert: './local.archive.org.cert',
+  sslKey: './local.archive.org.key',
+});

package/web-test-runner.config.mjs

@@ -1,41 +1,41 @@
-// import { playwrightLauncher } from '@web/test-runner-playwright';
-
-const filteredLogs = ['Running in dev mode', 'lit-html is in dev mode'];
-
-export default /** @type {import("@web/test-runner").TestRunnerConfig} */ ({
-  /** Test files to run */
-  files: 'dist/test/**/*.test.js',
-
-  /** Resolve bare module imports */
-  nodeResolve: {
-    exportConditions: ['browser', 'development'],
-  },
-
-  /** Filter out lit dev mode logs */
-  filterBrowserLogs(log) {
-    for (const arg of log.args) {
-      if (typeof arg === 'string' && filteredLogs.some(l => arg.includes(l))) {
-        return false;
-      }
-    }
-    return true;
-  },
-
-  /** Compile JS for older browsers. Requires @web/dev-server-esbuild plugin */
-  // esbuildTarget: 'auto',
-
-  /** Amount of browsers to run concurrently */
-  // concurrentBrowsers: 2,
-
-  /** Amount of test files per browser to test concurrently */
-  // concurrency: 1,
-
-  /** Browsers to run tests on */
-  // browsers: [
-  //   playwrightLauncher({ product: 'chromium' }),
-  //   playwrightLauncher({ product: 'firefox' }),
-  //   playwrightLauncher({ product: 'webkit' }),
-  // ],
-
-  // See documentation for all available options
-});
+// import { playwrightLauncher } from '@web/test-runner-playwright';
+
+const filteredLogs = ['Running in dev mode', 'lit-html is in dev mode'];
+
+export default /** @type {import("@web/test-runner").TestRunnerConfig} */ ({
+  /** Test files to run */
+  files: 'dist/test/**/*.test.js',
+
+  /** Resolve bare module imports */
+  nodeResolve: {
+    exportConditions: ['browser', 'development'],
+  },
+
+  /** Filter out lit dev mode logs */
+  filterBrowserLogs(log) {
+    for (const arg of log.args) {
+      if (typeof arg === 'string' && filteredLogs.some(l => arg.includes(l))) {
+        return false;
+      }
+    }
+    return true;
+  },
+
+  /** Compile JS for older browsers. Requires @web/dev-server-esbuild plugin */
+  // esbuildTarget: 'auto',
+
+  /** Amount of browsers to run concurrently */
+  // concurrentBrowsers: 2,
+
+  /** Amount of test files per browser to test concurrently */
+  // concurrency: 1,
+
+  /** Browsers to run tests on */
+  // browsers: [
+  //   playwrightLauncher({ product: 'chromium' }),
+  //   playwrightLauncher({ product: 'firefox' }),
+  //   playwrightLauncher({ product: 'webkit' }),
+  // ],
+
+  // See documentation for all available options
+});