fix thumbnail height/width #2030

Merged
neb-b merged 1 commit from thumbnail into master 2018-10-14 19:19:41 +02:00
neb-b commented 2018-10-12 00:58:07 +02:00 (Migrated from github.com)

Messed it up when I was refactoring the filecard

The thumbnail component is different from .card__media where it is a static height/width

Messed it up when I was refactoring the filecard The thumbnail component is different from `.card__media` where it is a static height/width
skhameneh (Migrated from github.com) approved these changes 2018-10-12 20:33:07 +02:00
skhameneh (Migrated from github.com) left a comment

Minor comment

Minor comment
@ -384,1 +395,4 @@
width: var(--thumbnail-preview-width);
background-size: cover;
background-repeat: no-repeat;
background-position: 50% 50%;
skhameneh (Migrated from github.com) commented 2018-10-12 20:32:58 +02:00

Is this line needed?

Is this line needed?
neb-b (Migrated from github.com) reviewed 2018-10-14 19:19:37 +02:00
@ -384,1 +395,4 @@
width: var(--thumbnail-preview-width);
background-size: cover;
background-repeat: no-repeat;
background-position: 50% 50%;
neb-b (Migrated from github.com) commented 2018-10-14 19:19:37 +02:00

Yeah. It makes sure the image is always centered.

Yeah. It makes sure the image is always centered.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#2030
No description provided.