From 399a633061577b8a2ca95c29ce1cfe0abeac4779 Mon Sep 17 00:00:00 2001
From: Timothy Andrew <mail@timothyandrew.net>
Date: Fri, 3 Jun 2016 09:07:37 +0530
Subject: [PATCH] Fix minor styling issues.

- No "Actions" label necessary
- `%td` can be moved out of `if/else`
- Page header should be "Profile Settings", not "Personal Access Tokens"
- "You don't have any tokens" message should be styled consistently
---
 .../personal_access_tokens/index.html.haml    | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/app/views/profiles/personal_access_tokens/index.html.haml b/app/views/profiles/personal_access_tokens/index.html.haml
index 503f88efb4..68f2813bc1 100644
--- a/app/views/profiles/personal_access_tokens/index.html.haml
+++ b/app/views/profiles/personal_access_tokens/index.html.haml
@@ -1,12 +1,11 @@
 - page_title "Personal Access Tokens"
-- header_title page_title, profile_personal_access_tokens_path
 
 .row.prepend-top-default
   .col-lg-3.profile-settings-sidebar
     %h4.prepend-top-0
       = page_title
     %p
-      You can generate a personal access token for each application you use that needs access to GitLab.
+      You can generate a personal access token for each application you use that needs access to the GitLab API.
   .col-lg-9
 
     - if flash[:personal_access_token]
@@ -49,21 +48,22 @@
               %th Name
               %th Created
               %th Expires
-              %th Actions
+              %th
           %tbody
             - @active_personal_access_tokens.each do |token|
               %tr
                 %td= token.name
                 %td= token.created_at.to_date.to_s(:medium)
-                - if token.expires_at.present?
-                  %td= token.expires_at.to_date.to_s(:medium)
-                - else
-                  %td
+                %td
+                  - if token.expires_at.present?
+                    = token.expires_at.to_date.to_s(:medium)
+                  - else
                     %span.personal-access-tokens-never-expires-label Never
                 %td= link_to "Revoke", revoke_profile_personal_access_token_path(token), method: :put, class: "btn btn-danger", data: { confirm: "Are you sure? This cannot be undone." }
 
     - else
-      %span You don't have any active tokens yet.
+      .settings-message.text-center
+        You don't have any active tokens yet.
 
     %hr
 
@@ -83,7 +83,8 @@
                 %td= token.created_at.to_date.to_s(:medium)
 
     - else
-      %span No inactive tokens.
+      .settings-message.text-center
+        There are no inactive tokens.
 
 
 :javascript