Skip to content

Commit 0ca249e

Browse files
committed
MOBILE-3401 site: Keep aspect ratio on site icons
1 parent 774b859 commit 0ca249e

File tree

2 files changed

+13
-3
lines changed

2 files changed

+13
-3
lines changed

src/core/login/pages/site/site.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ <h2 text-wrap>{{ 'core.login.yourenteredsite' | translate }}</h2>
3737
<div *ngIf="loadingSites" class="core-login-site-list-loading"><ion-spinner></ion-spinner></div>
3838
<button ion-item *ngFor="let site of sites" (click)="connect($event, site.url, site)" [attr.aria-label]="site.name" detail-push>
3939
<ion-thumbnail item-start>
40-
<img [src]="site.imageurl" *ngIf="site.imageurl">
40+
<img [src]="site.imageurl" *ngIf="site.imageurl" onError="this.src='assets/icon/icon.png'">
4141
<img src="assets/icon/icon.png" *ngIf="!site.imageurl" class="core-login-default-icon">
4242
</ion-thumbnail>
4343
<h2 text-wrap>{{site.name}}<ng-container *ngIf="site.alias"> ({{site.alias}})</ng-container></h2>

src/core/login/pages/site/site.scss

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,13 +27,23 @@ ion-app.app-root page-core-login-site {
2727
.item ion-thumbnail {
2828
min-width: 50px;
2929
min-height: 50px;
30+
width: 50px;
31+
height: 50px;
3032
border-radius: 20%;
3133
box-shadow: 0 0 4px #eee;
3234
text-align: center;
35+
overflow: hidden;
3336

3437
img {
35-
width: 50px;
36-
height: 50px;
38+
max-height: 50px;
39+
max-width: fit-content;
40+
width: auto;
41+
height: auto;
42+
margin: 0 auto;
43+
margin-left: 50%;
44+
transform: translateX(-50%);
45+
object-fit: cover;
46+
object-position: 50% 50%;
3747
}
3848
ion-icon {
3949
margin: 0 auto;

0 commit comments

Comments
 (0)