feat(web): improve and refactor thumbnails

[michelheusschen]

The following changes are made:

• Shrink the whole thumbnail when selected, except for the select button
• Improved animation when selecting thumbnail
• Allow video playback while the thumbnail is selected
• Provide methods on the API class for getting file and thumbnail URLs
• Show error icon when video loading fails
• Live photos would only play when hovering the top right icon, now plays when hovering anywhere (same as video)

And the following bugs are fixes:

• Selection border disappears when hovering video thumbnail, fixes [BUG] When selecting a video, the border disappears when hovering the video on web #1944
• Video duration with less than 10 seconds would appear as 0:5 instead of 0:05
• Video duration shows incorrectly for some duration before this was merged fix(server): invalid video duration format #2058
• Cancel video buffering when not hovering thumbnail anymore. Potentially fixes [BUG] [WEB] Video previews in Gridview keep downloading, even if they are not visible anymore (reopen) #958, might need validation

Currently there's some detection for horizontal and vertical asset orientation and sets the height/width accordingly. But I don't think exif data is actually present in a lot of cases and I'm not certain orientations like Rotate 90 CW are valid. Anyway, that still needs to be looked into further, but that's probably for a future PR.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated
immich	⬜️ Ignored (Inspect)			Mar 26, 2023 at 0:41AM (UTC)

[alextran1502]

Live photos would only play when hovering the top right icon, now plays when hovering anywhere (same as video)

This is the intended behavior for LivePhoto to play when hover on the icon

[michelheusschen]

Live photos would only play when hovering the top right icon, now plays when hovering anywhere (same as video)

This is the intended behavior for LivePhoto to play when hover on the icon

I understand, that's why I noted it as a change rather than a fix. Hovering over the icon to start playing live photos feels unintuitive to me, that's why I changed it. If you want however, it can be reverted.

[alextran1502]

Live photos would only play when hovering the top right icon, now plays when hovering anywhere (same as video)

This is the intended behavior for LivePhoto to play when hover on the icon

I understand, that's why I noted it as a change rather than a fix. Hovering over the icon to start playing live photos feels unintuitive to me, that's why I changed it. If you want however, it can be reverted.

Yes please, I would like to treat it as a normal photo, the motion part is extra and only run when you specifically want to

[jrasm91]

Looks good!