From d1992b93f0f4a4408f803d6320cd26a713e22d5b Mon Sep 17 00:00:00 2001
From: Chocobozzz <florian.bigard@gmail.com>
Date: Fri, 4 Nov 2016 16:04:50 +0100
Subject: [PATCH] Client: add more informations to watch video view

---
 client/src/app/videos/shared/video.model.ts   |  1 +
 client/src/app/videos/shared/video.service.ts |  1 +
 .../video-watch/video-watch.component.html    | 83 +++++++++++++++----
 .../video-watch/video-watch.component.scss    | 53 ++++++++++++
 .../video-watch/video-watch.component.ts      | 46 +++++-----
 5 files changed, 145 insertions(+), 39 deletions(-)

diff --git a/client/src/app/videos/shared/video.model.ts b/client/src/app/videos/shared/video.model.ts
index 65417f751..873c83ff1 100644
--- a/client/src/app/videos/shared/video.model.ts
+++ b/client/src/app/videos/shared/video.model.ts
@@ -21,6 +21,7 @@ export class Video {
       port = ':' + port;
     }
 
+
     return author + '@' + host + port;
   }
 
diff --git a/client/src/app/videos/shared/video.service.ts b/client/src/app/videos/shared/video.service.ts
index ad8557533..b1f688095 100644
--- a/client/src/app/videos/shared/video.service.ts
+++ b/client/src/app/videos/shared/video.service.ts
@@ -22,6 +22,7 @@ export class VideoService {
   getVideo(id: string): Observable<Video> {
     return this.http.get(VideoService.BASE_VIDEO_URL + id)
                     .map(this.restExtractor.extractDataGet)
+                    .map(video_hash => new Video(video_hash))
                     .catch((res) => this.restExtractor.handleError(res));
   }
 
diff --git a/client/src/app/videos/video-watch/video-watch.component.html b/client/src/app/videos/video-watch/video-watch.component.html
index ddabdc9e9..14947da88 100644
--- a/client/src/app/videos/video-watch/video-watch.component.html
+++ b/client/src/app/videos/video-watch/video-watch.component.html
@@ -1,23 +1,74 @@
-<div *ngIf="error" class="alert alert-danger">
-  The video load seems to be abnormally long. You could:
-  <ul>
-    <li>Check your browser console to see potentials errors</li>
-    <li>Your firewall or NAT could be too restrictive for WebRTC (there is no TURN server)</li>
-    <li>
-      Report an issue on
-      <a href="https://github.com/Chocobozzz/PeerTube/issues" title="Report an issue">
-        https://github.com/Chocobozzz/PeerTube/issues
-      </a>
-    </li>
-  </ul>
+<div *ngIf="error" class="row">
+  <div class="alert alert-danger">
+    The video load seems to be abnormally long.
+    <ul>
+      <li>Maybe the server {{ video.podUrl }} is down :(</li>
+      <li>
+        If not, you can report an issue on
+        <a href="https://github.com/Chocobozzz/PeerTube/issues" title="Report an issue">
+          https://github.com/Chocobozzz/PeerTube/issues
+        </a>
+      </li>
+    </ul>
+  </div>
 </div>
 
-<div class="embed-responsive embed-responsive-19by9">
-  <my-loader [loading]="loading"></my-loader>
+<div class="row">
+  <div class="col-md-12">
+    <div class="embed-responsive embed-responsive-19by9">
+      <my-loader [loading]="loading"></my-loader>
+    </div>
+  </div>
 </div>
 
-<div id="torrent-info">
+<div id="torrent-info" class="row">
   <div id="torrent-info-download" class="col-md-4 col-sm-4 col-xs-4">Download: {{ downloadSpeed | bytes }}/s</div>
   <div id="torrent-info-upload" class="col-md-4 col-sm-4 col-xs-4">Upload: {{ uploadSpeed | bytes }}/s</div>
   <div id="torrent-info-peers" class="col-md-4 col-sm-4 col-xs-4">Number of peers: {{ numPeers }}</div>
-<div>
+</div>
+
+<div *ngIf="video !== null" id="video-info">
+  <div class="row" id="video-name-actions">
+    <div class="col-md-8">
+      <div class="row">
+        <div id="video-name" class="col-md-12">
+          {{ video.name }}
+        </div>
+      </div>
+
+      <div class="row">
+        <div class="col-md-12" id="video-by-date">
+          <span id="video-by">
+              from
+              <a [routerLink]="['/videos/list', { field: 'author', search: video.author }]" class="video-miniature-author">
+                {{ video.by }}
+              </a>
+            </span>
+          <span id="video-date">on {{ video.createdDate | date:'short' }}</span>
+        </div>
+      </div>
+    </div>
+
+    <div id="video-actions" class="col-md-4 text-right">
+      <button title="Get magnet URI" id="magnet-uri" class="btn btn-default">
+        <span class="glyphicon glyphicon-magnet"></span> Magnet
+      </button>
+    </div>
+  </div>
+
+  <div id="video-tags" class="row">
+    <div class="col-md-12">
+      <a *ngFor="let tag of video.tags" [routerLink]="['/videos/list', { field: 'tags', search: tag }]" class="label label-primary">
+        {{ tag }}
+      </a>
+    </div>
+  </div>
+
+  <div id="video-description" class="row">
+    <div class="col-md-12">
+      <div id="description-label">Description</div>
+      {{ video.description }}
+    </div>
+  </div>
+</div>
+
diff --git a/client/src/app/videos/video-watch/video-watch.component.scss b/client/src/app/videos/video-watch/video-watch.component.scss
index 96420582a..e1d9300cd 100644
--- a/client/src/app/videos/video-watch/video-watch.component.scss
+++ b/client/src/app/videos/video-watch/video-watch.component.scss
@@ -24,3 +24,56 @@ my-loader {
     height: 300px;
   }
 }
+
+#video-info {
+  border-top: 1px solid rgba(0, 0, 0, 0.1);
+  margin-top: 10px;
+  padding-top: 5px;
+
+  #video-name-actions {
+
+    #video-name {
+      font-size: 20px;
+    }
+
+    .btn .glyphicon {
+      position: relative;
+      top: 2px;
+    }
+
+    #magnet-uri {
+      font-weight: bold;
+      opacity: 0.85;
+    }
+  }
+
+  #video-by-date {
+    font-size: 13px;
+    opacity: 0.6;
+
+    a {
+      color: black;
+    }
+
+    #video-date:before {
+      content: '\002022';
+      margin: 0 5px;
+    }
+  }
+
+  #video-tags {
+    margin-top: 10px;
+
+    a {
+      margin-right: 5px;
+    }
+  }
+
+  #video-description {
+    margin-top: 15px;
+
+    #description-label {
+      font-weight: bold;
+    }
+  }
+}
diff --git a/client/src/app/videos/video-watch/video-watch.component.ts b/client/src/app/videos/video-watch/video-watch.component.ts
index 239e24c99..9a36c17e2 100644
--- a/client/src/app/videos/video-watch/video-watch.component.ts
+++ b/client/src/app/videos/video-watch/video-watch.component.ts
@@ -18,7 +18,7 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
   loading: boolean = false;
   numPeers: number;
   uploadSpeed: number;
-  video: Video;
+  video: Video = null;
 
   private errorTimer: NodeJS.Timer;
   private sub: any;
@@ -32,6 +32,28 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
     private webTorrentService: WebTorrentService
   ) {}
 
+  ngOnInit() {
+    this.sub = this.route.params.subscribe(routeParams => {
+      let id = routeParams['id'];
+      this.videoService.getVideo(id).subscribe(
+        video => {
+          this.video = video;
+          this.loadVideo();
+        },
+        error => alert(error.text)
+      );
+    });
+  }
+
+  ngOnDestroy() {
+    console.log('Removing video from webtorrent.');
+    clearInterval(this.torrentInfosInterval);
+    clearTimeout(this.errorTimer);
+    this.webTorrentService.remove(this.video.magnetUri);
+
+    this.sub.unsubscribe();
+  }
+
   loadVideo() {
     // Reset the error
     this.error = false;
@@ -65,28 +87,6 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
     });
   }
 
-  ngOnDestroy() {
-    console.log('Removing video from webtorrent.');
-    clearInterval(this.torrentInfosInterval);
-    clearTimeout(this.errorTimer);
-    this.webTorrentService.remove(this.video.magnetUri);
-
-    this.sub.unsubscribe();
-  }
-
-  ngOnInit() {
-    this.sub = this.route.params.subscribe(routeParams => {
-      let id = routeParams['id'];
-      this.videoService.getVideo(id).subscribe(
-        video => {
-          this.video = video;
-          this.loadVideo();
-        },
-        error => alert(error.text)
-      );
-    });
-  }
-
   private loadTooLong() {
     this.error = true;
     console.error('The video load seems to be abnormally long.');
-- 
2.41.0