RTL Support in Emails

app/views/notify/_note_message.html.haml

 %div
   = markdown(@note.note)
+
+:javascript
+  document.addEventListener("DOMContentLoaded", function(event) {
+    var elements = document.getElementsByTagName("*");
+    for (var i = 0; i < elements.length; ++i) {
+      elements[i].setAttribute("dir", "auto");
+    }
+  });

app/views/notify/_reassigned_issuable_email.html.haml

@@ -2,9 +2,9 @@
   Assignee changed
   - if @previous_assignee
     from
-    %strong #{@previous_assignee.name}
+    %strong{ dir: :auto } #{@previous_assignee.name}
   to
   - if issuable.assignee_id
-    %strong #{issuable.assignee_name}
+    %strong{ dir: :auto } #{issuable.assignee_name}
   - else
     %strong Unassigned

app/views/notify/closed_issue_email.html.haml

 %p
-  = "Issue was closed by #{@updated_by.name}"
+  = "Issue was closed by "
+  %span{ dir: :auto }= @updated_by.name

app/views/notify/closed_merge_request_email.html.haml

 %p
-  = "Merge Request ##{@merge_request.iid} was closed by #{@updated_by.name}"
+  = "Merge Request ##{@merge_request.iid} was closed by "
+  %span{ dir: :auto }= @updated_by.name

app/views/notify/group_access_granted_email.html.haml

 %p
   = "You have been granted #{@membership.human_access} access to group"
   = link_to group_url(@group) do
-    = @group.name
+    %span{ dir: :auto }= @group.name

app/views/notify/issue_status_changed_email.html.haml

 %p
-  = "Issue was #{@issue_status} by #{@updated_by.name}"
+  = "Issue was #{@issue_status} by "
+  %span{ dir: :auto }= @updated_by.name

app/views/notify/merge_request_status_email.html.haml

 %p
-  = "Merge Request ##{@merge_request.iid} was #{@mr_status} by #{@updated_by.name}"
+  = "Merge Request ##{@merge_request.iid} was #{@mr_status} by "
+  %span{ dir: :auto }= @updated_by.name

app/views/notify/new_email_email.html.haml

 %p
-  Hi #{@user.name}!
+  Hi
+  %span{ dir: :auto }= @user.name
+  !
 %p
   A new email was added to your account:
 %p

app/views/notify/new_issue_email.html.haml

@@ -3,4 +3,13 @@
 
 - if @issue.assignee_id.present?
   %p
-    Assignee: #{@issue.assignee_name}
+    Assignee:
+    %span= @issue.assignee_name
+
+:javascript
+  document.addEventListener("DOMContentLoaded", function(event) {
+    var elements = document.getElementsByTagName("*");
+    for (var i = 0; i < elements.length; ++i) {
+      elements[i].setAttribute("dir", "auto");
+    }
+  });

app/views/notify/new_merge_request_email.html.haml

@@ -3,7 +3,18 @@
 
 - if @merge_request.assignee_id.present?
   %p
-    Assignee: #{@merge_request.author_name} → #{@merge_request.assignee_name}
+    Assignee:
+    %span= @merge_request.author_name
+    →
+    %span= @merge_request.assignee_name
 
 -if @merge_request.description
   = markdown(@merge_request.description)
+
+:javascript
+  document.addEventListener("DOMContentLoaded", function(event) {
+    var elements = document.getElementsByTagName("*");
+    for (var i = 0; i < elements.length; ++i) {
+      elements[i].setAttribute("dir", "auto");
+    }
+  });

app/views/notify/new_ssh_key_email.html.haml

 %p
-  Hi #{@user.name}!
+  Hi
+  %span{ dir: :auto }= @user.name
+  !
 %p
   A new public key was added to your account:
 %p
   title:
-  %code= @key.title
+  %code{ dir: :auto }= @key.title
 %p
   If this key was added in error, you can remove it here:
   = link_to "SSH Keys", profile_keys_url

app/views/notify/new_user_email.html.haml

 %p
-  Hi #{@user['name']}!
+  Hi
+  %span{ dir: :auto }= @user['name']
+  !
 %p
   - if Gitlab.config.gitlab.signup_enabled
     Your account has been created successfully.

app/views/notify/project_access_granted_email.html.haml

@@ -2,4 +2,4 @@
   = "You have been granted #{@project_member.human_access} access to project"
 %p
   = link_to project_url(@project) do
-    = @project.name_with_namespace
+    %span{ dir: :auto }= @project.name_with_namespace

app/views/notify/project_was_moved_email.html.haml

@@ -3,7 +3,7 @@
 %p
   The project is now located under
   = link_to project_url(@project) do
-    = @project.name_with_namespace
+    %span{ dir: :auto }= @project.name_with_namespace
 %p
   To update the remote url in your local repository run (for ssh):
 %p{ style: "background:#f5f5f5; padding:10px; border:1px solid #ddd" }

app/views/notify/repository_push_email.html.haml

@@ -26,3 +26,11 @@
 
 - if @compare.timeout
   %h5 Huge diff. To prevent performance issues changes are hidden
+
+:javascript
+  document.addEventListener("DOMContentLoaded", function(event) {
+    var elements = document.getElementsByTagName("*");
+    for (var i = 0; i < elements.length; ++i) {
+      elements[i].setAttribute("dir", "auto");
+    }
+  });