From 80c7779efeab644410dee2f21428322b1f0e3500 Mon Sep 17 00:00:00 2001
From: Rigel Kent <sendmemail@rigelk.eu>
Date: Thu, 4 Jun 2020 20:55:25 +0200
Subject: fix mascot sizes, fix tick, delete extraneous .png icons

follows PR #2822
---
 .../my-account-video-playlist-edit.component.html            |  8 ++++----
 .../app/modal/instance-config-warning-modal.component.html   | 12 +++++-------
 .../app/modal/instance-config-warning-modal.component.scss   | 10 ++--------
 client/src/app/modal/welcome-modal.component.html            |  8 ++------
 client/src/app/modal/welcome-modal.component.scss            |  6 ++++++
 client/src/app/shared/video/abstract-video-list.scss         |  1 -
 6 files changed, 19 insertions(+), 26 deletions(-)

(limited to 'client/src/app')

diff --git a/client/src/app/+my-account/my-account-video-playlists/my-account-video-playlist-edit.component.html b/client/src/app/+my-account/my-account-video-playlists/my-account-video-playlist-edit.component.html
index 05335dc1a..ff9cd37ea 100644
--- a/client/src/app/+my-account/my-account-video-playlists/my-account-video-playlist-edit.component.html
+++ b/client/src/app/+my-account/my-account-video-playlists/my-account-video-playlist-edit.component.html
@@ -32,7 +32,7 @@
         <div class="form-group">
           <label i18n for="displayName">Display name</label>
           <input
-            type="text" id="displayName"
+            type="text" id="displayName" class="form-control"
             formControlName="displayName" [ngClass]="{ 'input-error': formErrors['displayName'] }"
           >
           <div *ngIf="formErrors['displayName']" class="form-error">
@@ -44,7 +44,7 @@
           <label i18n for="description">Description</label>
           <textarea
             id="description" formControlName="description"
-            [ngClass]="{ 'input-error': formErrors['description'] }"
+            class="form-control" [ngClass]="{ 'input-error': formErrors['description'] }"
           ></textarea>
           <div *ngIf="formErrors.description" class="form-error">
             {{ formErrors.description }}
@@ -56,7 +56,7 @@
         <div class="form-group">
           <label i18n for="privacy">Privacy</label>
           <div class="peertube-select-container">
-            <select id="privacy" formControlName="privacy">
+            <select id="privacy" formControlName="privacy" class="form-control">
               <option *ngFor="let privacy of videoPlaylistPrivacies" [value]="privacy.id">{{ privacy.label }}</option>
             </select>
           </div>
@@ -69,7 +69,7 @@
         <div class="form-group">
           <label i18n>Channel</label>
           <div class="peertube-select-container">
-            <select formControlName="videoChannelId">
+            <select formControlName="videoChannelId" class="form-control">
               <option></option>
               <option *ngFor="let channel of userVideoChannels" [value]="channel.id">{{ channel.label }}</option>
             </select>
diff --git a/client/src/app/modal/instance-config-warning-modal.component.html b/client/src/app/modal/instance-config-warning-modal.component.html
index 5a8290cda..5a8adf726 100644
--- a/client/src/app/modal/instance-config-warning-modal.component.html
+++ b/client/src/app/modal/instance-config-warning-modal.component.html
@@ -7,7 +7,7 @@
   <div class="modal-body">
     <img class="mascot" src="/client/assets/images/mascot/oh.svg" alt="mascot" />
 
-    <p i18n>Hello dear administrator. You enabled user registration on your instance but you did not configure the following fields:</p>
+    <p i18n>You enabled user registration on your instance but did not configure the following fields:</p>
 
     <ul>
       <li i18n *ngIf="about.instance.name.toLowerCase() === 'peertube'">Instance name</li>
@@ -25,11 +25,6 @@
       Please consider configuring these fields to help people to choose <strong>the appropriate instance</strong>.
       Without them, your instance may not be referenced on the <a target="_blank" rel="noopener noreferrer" href="https://joinpeertube.org">JoinPeerTube website</a>.
     </p>
-
-    <div class="configure-instance">
-      <a i18n class="action-button action-button-configure" href="/admin/config/edit-custom" target="_blank" rel="noopener noreferrer">Configure these fields</a>
-    </div>
-
   </div>
 
   <div class="modal-footer inputs">
@@ -37,13 +32,16 @@
       inputName="stopDisplayModal" [(ngModel)]="stopDisplayModal"
       i18n-labelText labelText="Don't show me this warning anymore"
     >
-
     </my-peertube-checkbox>
 
     <input
       type="button" role="button" i18n-value value="Close" class="action-button action-button-cancel"
       (click)="hide()" (key.enter)="hide()"
     >
+    <a i18n class="action-button action-button-configure" ngbAutofocus
+       href="/admin/config/edit-custom" target="_blank" rel="noopener noreferrer">
+       Configure
+    </a>
   </div>
 
 </ng-template>
diff --git a/client/src/app/modal/instance-config-warning-modal.component.scss b/client/src/app/modal/instance-config-warning-modal.component.scss
index 4a76a1319..cc97d64e4 100644
--- a/client/src/app/modal/instance-config-warning-modal.component.scss
+++ b/client/src/app/modal/instance-config-warning-modal.component.scss
@@ -13,17 +13,10 @@ li {
   margin-bottom: 10px;
 }
 
-.configure-instance {
-  text-align: center;
-  font-weight: 600;
-  font-size: 18px;
-  margin-top: 40px;
-  margin-bottom: 10px;
-}
-
 .mascot {
   display: block;
   margin: 0 auto 50px;
+  width: 25%;
 }
 
 .action-button-configure {
@@ -31,4 +24,5 @@ li {
 
   @include peertube-button;
   @include orange-button;
+  @include disable-default-a-behaviour;
 }
diff --git a/client/src/app/modal/welcome-modal.component.html b/client/src/app/modal/welcome-modal.component.html
index 8bfcc4bf6..81c12d4c0 100644
--- a/client/src/app/modal/welcome-modal.component.html
+++ b/client/src/app/modal/welcome-modal.component.html
@@ -33,9 +33,7 @@
 
     <div class="two-columns">
 
-      <div class="mascot">
-        <img src="/client/assets/images/mascot/pointing.png" alt="mascot">
-      </div>
+      <img class="mascot mascot-fw" src="/client/assets/images/mascot/pointing.svg" alt="mascot">
 
       <div class="block-links">
         <div i18n class="subtitle">Useful links</div>
@@ -51,9 +49,7 @@
     </div>
 
     <div class="two-columns">
-      <div class="mascot">
-        <img src="/client/assets/images/mascot/happy.png" alt="mascot">
-      </div>
+      <img class="mascot" src="/client/assets/images/mascot/happy.svg" alt="mascot">
 
       <div class="block-configuration">
         <div i18n class="subtitle">It's time to configure your instance!</div>
diff --git a/client/src/app/modal/welcome-modal.component.scss b/client/src/app/modal/welcome-modal.component.scss
index 6214f3b9d..c04772546 100644
--- a/client/src/app/modal/welcome-modal.component.scss
+++ b/client/src/app/modal/welcome-modal.component.scss
@@ -12,9 +12,14 @@
   margin-top: 50px;
 }
 
+.mascot-fw {
+  width: 170px;
+}
+
 .mascot {
   display: block;
   min-width: 170px;
+  margin-right: 2rem;
 }
 
 .subtitle {
@@ -73,6 +78,7 @@ li {
 .configure-instance-button {
   @include peertube-button;
   @include orange-button;
+  @include disable-default-a-behaviour;
 
   display: inline-block;
 }
diff --git a/client/src/app/shared/video/abstract-video-list.scss b/client/src/app/shared/video/abstract-video-list.scss
index 44b629542..4149c9109 100644
--- a/client/src/app/shared/video/abstract-video-list.scss
+++ b/client/src/app/shared/video/abstract-video-list.scss
@@ -4,7 +4,6 @@
 .videos-header {
   display: flex;
   justify-content: space-between;
-  height: 80px;
   align-items: baseline;
 
   .title-page.title-page-single {
-- 
cgit v1.2.3