Bidi support

5 files changed, 55 insertions, 21 deletions

diff --git a/client/src/app/+videos/+video-watch/comment/video-comment-add.component.html b/client/src/app/+videos/+video-watch/comment/video-comment-add.component.html
index 42adfed8d..3ee818c8b 100644
--- a/client/src/app/+videos/+video-watch/comment/video-comment-add.component.html
+++ b/client/src/app/+videos/+video-watch/comment/video-comment-add.component.html
@@ -10,7 +10,10 @@
                 (keyup.control.enter)="onValidKey()" (keyup.meta.enter)="onValidKey()" #textarea>
       </textarea>
 
-      <my-help class="markdown-guide" helpType="custom" iconName="markdown" tooltipPlacement="left auto" autoClose="true" i18n-title title="Markdown compatible">
+      <my-help
+        [ngClass]="{ 'is-rtl': isRTL() }" class="markdown-guide" helpType="custom" iconName="markdown"
+        tooltipPlacement="left auto" autoClose="true" i18n-title title="Markdown compatible"
+      >
         <ng-template ptTemplate="customHtml">
           <span i18n>Markdown compatible that supports:</span>
 
diff --git a/client/src/app/+videos/+video-watch/comment/video-comment-add.component.scss b/client/src/app/+videos/+video-watch/comment/video-comment-add.component.scss
index 7743bd41d..5c97fb13c 100644
--- a/client/src/app/+videos/+video-watch/comment/video-comment-add.component.scss
+++ b/client/src/app/+videos/+video-watch/comment/video-comment-add.component.scss
@@ -14,7 +14,7 @@ form {
   margin-bottom: 10px;
 
   my-actor-avatar {
-    margin-right: 10px;
+    @include margin-right(10px);
   }
 
   .form-group {
@@ -26,12 +26,12 @@ form {
   textarea {
     @include peertube-textarea(100%, $peertube-textarea-height);
     @include button-focus(pvar(--mainColorLightest));
+    @include padding-right($markdown-icon-width + 15px !important);
 
     min-height: calc(#{$peertube-textarea-height} - 15px * 2);
-    padding-right: $markdown-icon-width + 15px !important;
 
     @media screen and (max-width: 600px) {
-      padding-right: $markdown-icon-width + 19px !important;
+      @include padding-right($markdown-icon-width + 19px !important);
     }
 
     &:focus::placeholder {
@@ -45,6 +45,12 @@ form {
   top: 5px;
   right: 9px;
 
+  // inset-inline is not well supported by web browsers
+  &.is-rtl {
+    right: unset;
+    left: 9px;
+  }
+
   ::ng-deep .help-tooltip-button {
     my-global-icon {
       height: $markdown-icon-height;
@@ -86,9 +92,10 @@ form {
     flex: 1;
 
     code {
+      @include margin-left(5px);
+
       display: inline-block;
       vertical-align: middle;
-      margin-left: 5px;
     }
   }
 }
diff --git a/client/src/app/+videos/+video-watch/comment/video-comment-add.component.ts b/client/src/app/+videos/+video-watch/comment/video-comment-add.component.ts
index 0e1362ad3..c926d8d70 100644
--- a/client/src/app/+videos/+video-watch/comment/video-comment-add.component.ts
+++ b/client/src/app/+videos/+video-watch/comment/video-comment-add.component.ts
@@ -1,5 +1,18 @@
 import { Observable } from 'rxjs'
-import { Component, ElementRef, EventEmitter, Input, OnChanges, OnInit, Output, SimpleChanges, ViewChild } from '@angular/core'
+import { getLocaleDirection } from '@angular/common'
+import {
+  Component,
+  ElementRef,
+  EventEmitter,
+  Inject,
+  Input,
+  LOCALE_ID,
+  OnChanges,
+  OnInit,
+  Output,
+  SimpleChanges,
+  ViewChild
+} from '@angular/core'
 import { Router } from '@angular/router'
 import { Notifier, User } from '@app/core'
 import { VIDEO_COMMENT_TEXT_VALIDATOR } from '@app/shared/form-validators/video-comment-validators'
@@ -37,7 +50,8 @@ export class VideoCommentAddComponent extends FormReactive implements OnChanges,
     private notifier: Notifier,
     private videoCommentService: VideoCommentService,
     private modalService: NgbModal,
-    private router: Router
+    private router: Router,
+    @Inject(LOCALE_ID) private localeId: string
   ) {
     super()
   }
@@ -153,6 +167,10 @@ export class VideoCommentAddComponent extends FormReactive implements OnChanges,
     this.form.value['text'] = this.textareaElement.nativeElement.value = ''
   }
 
+  isRTL () {
+    return getLocaleDirection(this.localeId) === 'rtl'
+  }
+
   private addCommentReply (commentCreate: VideoCommentCreate) {
     return this.videoCommentService
       .addCommentReply(this.video.id, this.parentComment.id, commentCreate)
diff --git a/client/src/app/+videos/+video-watch/comment/video-comment.component.scss b/client/src/app/+videos/+video-watch/comment/video-comment.component.scss
index a4d2e237c..7868991ba 100644
--- a/client/src/app/+videos/+video-watch/comment/video-comment.component.scss
+++ b/client/src/app/+videos/+video-watch/comment/video-comment.component.scss
@@ -6,10 +6,11 @@
   display: flex;
 
   .left {
+    @include margin-right(10px);
+
     display: flex;
     flex-direction: column;
     align-items: center;
-    margin-right: 10px;
 
     .vertical-border {
       width: 2px;
@@ -51,6 +52,9 @@ my-actor-avatar {
 }
 
 .video-author {
+  @include padding-right(6px);
+  @include padding-left(6px);
+
   height: 20px;
   background-color: #888888;
   border-radius: 12px;
@@ -60,8 +64,6 @@ my-actor-avatar {
   flex-direction: row;
   align-items: center;
   display: inline-flex;
-  padding-right: 6px;
-  padding-left: 6px;
   color: #fff !important;
 }
 
@@ -86,9 +88,10 @@ my-actor-avatar {
 }
 
 .comment-date {
+  @include margin-left(5px);
+
   font-size: 90%;
   color: pvar(--greyForegroundColor);
-  margin-left: 5px;
   text-decoration: none;
 
   &:hover {
@@ -129,9 +132,10 @@ my-actor-avatar {
 
   ::ng-deep .dropdown-toggle,
   .comment-action-reply {
+    @include margin-right(10px);
+
     color: pvar(--greyForegroundColor);
     cursor: pointer;
-    margin-right: 10px;
 
     &:hover,
     &:active,
@@ -162,25 +166,25 @@ my-video-comment-add {
   }
 
   .left {
-    margin-right: 6px;
+    @include margin-right(6px);
   }
 }
 
 @media screen and (max-width: 1200px) {
   .children {
-    margin-left: -10px;
+    @include margin-left(-10px);
   }
 }
 
 @media screen and (max-width: 600px) {
   .children {
-    margin-left: -20px;
+    @include margin-left(-20px);
 
     .left {
       align-items: flex-start;
 
       .vertical-border {
-        margin-left: 2px;
+        @include margin-left(2px);
       }
     }
   }
@@ -189,7 +193,7 @@ my-video-comment-add {
     flex-direction: column;
 
     .comment-date {
-      margin-left: 0;
+      @include margin-left(0);
     }
   }
 }
diff --git a/client/src/app/+videos/+video-watch/comment/video-comments.component.scss b/client/src/app/+videos/+video-watch/comment/video-comments.component.scss
index a7e858069..29a00ee3f 100644
--- a/client/src/app/+videos/+video-watch/comment/video-comments.component.scss
+++ b/client/src/app/+videos/+video-watch/comment/video-comments.component.scss
@@ -13,19 +13,21 @@
 
 .glyphicon,
 .comment-thread-loading {
-  margin-right: 5px;
+  @include margin-right(5px);
+
   display: inline-block;
   font-size: 13px;
 }
 
 .title-block {
   .title-page {
-    margin-right: 0;
+    @include margin-right(0);
   }
 
   my-feed {
+    @include margin-left(5px);
+
     display: inline-block;
-    margin-left: 5px;
     opacity: 0;
     transition: ease-in .2s opacity;
     width: 12px;
@@ -47,7 +49,7 @@
 
 @media screen and (max-width: 600px) {
   .view-replies {
-    margin-left: 46px;
+    @include margin-left(46px);
   }
 }