Skip to content

Commit 283b345

Browse files
committed
always use thumbnail component for files not supported by the media viewer, and switch to themed version of thumbnail component
1 parent 70d1e49 commit 283b345

File tree

2 files changed

+4
-9
lines changed

2 files changed

+4
-9
lines changed

src/app/item-page/media-viewer/media-viewer.component.html

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,7 @@
1616
</ng-container>
1717
</div>
1818
<ng-template #showThumbnail>
19-
<ds-themed-media-viewer-image *ngIf="mediaOptions.image && mediaOptions.video"
20-
[image]="(thumbnailsRD$ | async)?.payload?.page[0]?._links.content.href || thumbnailPlaceholder"
21-
[preview]="false"
22-
></ds-themed-media-viewer-image>
23-
<ds-thumbnail *ngIf="!(mediaOptions.image && mediaOptions.video)"
24-
[thumbnail]="(thumbnailsRD$ | async)?.payload?.page[0]">
25-
</ds-thumbnail>
19+
<ds-themed-thumbnail [thumbnail]="(thumbnailsRD$ | async)?.payload?.page[0]">
20+
</ds-themed-thumbnail>
2621
</ng-template>
2722
</ng-container>

src/app/item-page/media-viewer/media-viewer.component.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -139,9 +139,9 @@ describe('MediaViewerComponent', () => {
139139
expect(mediaItem.thumbnail).toBe(null);
140140
});
141141

142-
it('should display a default, thumbnail', () => {
142+
it('should display a default thumbnail', () => {
143143
const defaultThumbnail = fixture.debugElement.query(
144-
By.css('ds-themed-media-viewer-image')
144+
By.css('ds-themed-thumbnail')
145145
);
146146
expect(defaultThumbnail.nativeElement).toBeDefined();
147147
});

0 commit comments

Comments
 (0)