Merge branch 'full-width-tables' into 'master'

Make tables full width. ## Before ![Screen_Shot_2015-10-19_at_11.17.32](/uploads/acea66825b41006a949b5cc31cc07699/Screen_Shot_2015-10-19_at_11.17.32.png) ## After ![Screen_Shot_2015-10-19_at_11.19.16](/uploads/ed61e5e6cafe06c5e2d75672f5585c93/Screen_Shot_2015-10-19_at_11.19.16.png) See merge request !1642

Merge branch 'full-width-tables' into 'master'
Make tables full width. ## Before ![Screen_Shot_2015-10-19_at_11.17.32](/uploads/acea66825b41006a949b5cc31cc07699/Screen_Shot_2015-10-19_at_11.17.32.png) ## After ![Screen_Shot_2015-10-19_at_11.19.16](/uploads/ed61e5e6cafe06c5e2d75672f5585c93/Screen_Shot_2015-10-19_at_11.19.16.png) See merge request !1642
5f045df7 · Dmitriy Zaporozhets · 35618a3d · 0f2b10c6 · 5f045df7 · 5f045df7
Commit 5f045df7 authored Oct 21, 2015 by Dmitriy Zaporozhets
41 changed files
--- a/app/assets/stylesheets/framework/files.scss
+++ b/app/assets/stylesheets/framework/files.scss
@@ -98,7 +98,6 @@
          border-right: none;
        }
        background: #fff;
-        padding: 10px $gl-padding;
      }
      .lines {
        pre {

--- a/app/assets/stylesheets/framework/lists.scss
+++ b/app/assets/stylesheets/framework/lists.scss
@@ -107,7 +107,7 @@ ul.content-list {
  > li {
    padding: $gl-padding;
-    border-color: #f1f2f4;
+    border-color: $table-border-color;
    margin-left: -$gl-padding;
    margin-right: -$gl-padding;
    color: $gl-gray;

--- a/app/assets/stylesheets/framework/tables.scss
+++ b/app/assets/stylesheets/framework/tables.scss
+.table-holder {
+  margin: -$gl-padding;
+  margin-top: 0;
+  margin-bottom: 0;
+}
 table {
  &.table {
    .dropdown-menu a {
@@ -18,15 +24,17 @@ table {
    tr {
      td, th {
-        padding: 8px 10px;
+        padding: 10px $gl-padding;
        line-height: 20px;
        vertical-align: middle;
      }
      th {
        font-weight: normal;
        font-size: 15px;
        border-bottom: 1px solid $border-color !important;
      }
      td {
        border-color: $table-border-color !important;
        border-bottom: 1px solid;

--- a/app/assets/stylesheets/framework/timeline.scss
+++ b/app/assets/stylesheets/framework/timeline.scss
@@ -6,7 +6,7 @@
  .timeline-entry {
    padding: $gl-padding;
-    border-color: #f1f2f4;
+    border-color: $table-border-color;
    margin-left: -$gl-padding;
    margin-right: -$gl-padding;
    color: $gl-gray;

--- a/app/assets/stylesheets/pages/ci_projects.scss
+++ b/app/assets/stylesheets/pages/ci_projects.scss
@@ -6,11 +6,6 @@
    line-height: 1.5;
  }
-  .wide-table-holder {
-    margin-left: -$gl-padding;
-    margin-right: -$gl-padding;
-  }
  .builds,
  .projects-table {
    .light {

--- a/app/assets/stylesheets/pages/events.scss
+++ b/app/assets/stylesheets/pages/events.scss
@@ -7,7 +7,7 @@
  padding: $gl-padding;
  margin-left: -$gl-padding;
  margin-right: -$gl-padding;
-  border-bottom: 1px solid #f1f2f4;
+  border-bottom: 1px solid $table-border-color;
  color: #7f8fa4;
  &.event-inline {

--- a/app/assets/stylesheets/pages/projects.scss
+++ b/app/assets/stylesheets/pages/projects.scss
@@ -457,7 +457,7 @@ pre.light-well {
  .project-row {
    padding: $gl-padding;
-    border-color: #f1f2f4;
+    border-color: $table-border-color;
    margin-left: -$gl-padding;
    margin-right: -$gl-padding;

--- a/app/assets/stylesheets/pages/tree.scss
+++ b/app/assets/stylesheets/pages/tree.scss
 .tree-holder {
-  .tree-table-holder {
-    margin-left: -$gl-padding;
-    margin-right: -$gl-padding;
-  }
  .tree-table {
    margin-bottom: 0;
    tr {
      > td, > th {
-        padding: 10px $gl-padding;
        line-height: 32px;
-        border-color: $table-border-color !important;
      }
      &:hover {

--- a/app/views/admin/abuse_reports/index.html.haml
+++ b/app/views/admin/abuse_reports/index.html.haml
@@ -2,6 +2,7 @@
 %h3.page-title Abuse Reports
 %hr
 - if @abuse_reports.present?
+  .table-holder
    %table.table
      %thead
        %tr

--- a/app/views/admin/applications/show.html.haml
+++ b/app/views/admin/applications/show.html.haml
@@ -3,7 +3,8 @@
  Application: #{@application.name}
-%table.table
+.table-holder
+  %table.table
    %tr
      %td
        Application Id

--- a/app/views/admin/background_jobs/show.html.haml
+++ b/app/views/admin/background_jobs/show.html.haml
@@ -12,6 +12,7 @@
        %i.fa.fa-exclamation-triangle
        There are no running sidekiq processes. Please restart GitLab
    - else
+      .table-holder
        %table.table
          %thead
            %th USER

--- a/app/views/admin/deploy_keys/index.html.haml
+++ b/app/views/admin/deploy_keys/index.html.haml
@@ -5,6 +5,7 @@
    .panel-head-actions
      = link_to 'New Deploy Key', new_admin_deploy_key_path, class: "btn btn-new btn-sm"
  - if @deploy_keys.any?
+    .table-holder
      %table.table
        %thead.panel-heading
          %tr

--- a/app/views/admin/identities/index.html.haml
+++ b/app/views/admin/identities/index.html.haml
@@ -2,6 +2,7 @@
 = render 'admin/users/head'
 - if @identities.present?
+  .table-holder
    %table.table
      %thead
        %tr

--- a/app/views/admin/services/index.html.haml
+++ b/app/views/admin/services/index.html.haml
@@ -2,7 +2,8 @@
 %h3.page-title Service templates
 %p.light Service template allows you to set default values for project services
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th

--- a/app/views/ci/admin/events/index.html.haml
+++ b/app/views/ci/admin/events/index.html.haml
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th User ID

--- a/app/views/ci/admin/projects/index.html.haml
+++ b/app/views/ci/admin/projects/index.html.haml
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th ID

--- a/app/views/ci/admin/runners/index.html.haml
+++ b/app/views/ci/admin/runners/index.html.haml
@@ -35,7 +35,8 @@
 %br
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th Type

--- a/app/views/ci/events/index.html.haml
+++ b/app/views/ci/events/index.html.haml
 %h3.page-title Events
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th User ID

--- a/app/views/ci/lints/_create.html.haml
+++ b/app/views/ci/lints/_create.html.haml
@@ -4,6 +4,7 @@
    syntax is correct
    %i.fa.fa-ok.correct-syntax
+  .table-holder
    %table.table.table-bordered
      %thead
        %tr

--- a/app/views/dashboard/milestones/show.html.haml
+++ b/app/views/dashboard/milestones/show.html.haml
@@ -13,7 +13,9 @@
    %span All issues for this milestone are closed. You may close the milestone now.
 .description
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th Project

--- a/app/views/doorkeeper/applications/index.html.haml
+++ b/app/views/doorkeeper/applications/index.html.haml
 - page_title "Applications"
 %h3.page-title Your applications
 %p= link_to 'New Application', new_oauth_application_path, class: 'btn btn-success'
-%table.table.table-striped
+.table-holder
+  %table.table.table-striped
    %thead
      %tr
        %th Name

--- a/app/views/doorkeeper/applications/show.html.haml
+++ b/app/views/doorkeeper/applications/show.html.haml
@@ -2,8 +2,8 @@
 %h3.page-title
  Application: #{@application.name}
+.table-holder
-%table.table
+  %table.table
    %tr
      %td
        Application Id

--- a/app/views/doorkeeper/authorized_applications/index.html.haml
+++ b/app/views/doorkeeper/authorized_applications/index.html.haml
 %header.page-header
  %h1 Your authorized applications
 %main{:role => "main"}
+  .table-holder
    %table.table.table-striped
      %thead
        %tr

--- a/app/views/groups/milestones/show.html.haml
+++ b/app/views/groups/milestones/show.html.haml
@@ -21,7 +21,9 @@
    %span All issues for this milestone are closed. You may close the milestone now.
 .description
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th Project

--- a/app/views/import/bitbucket/status.html.haml
+++ b/app/views/import/bitbucket/status.html.haml
@@ -14,7 +14,8 @@
    = button_tag 'Import all projects', class: "btn btn-success js-import-all"
-%table.table.import-jobs
+.table-holder
+  %table.table.import-jobs
    %thead
      %tr
        %th From Bitbucket

--- a/app/views/import/fogbugz/new_user_map.html.haml
+++ b/app/views/import/fogbugz/new_user_map.html.haml
@@ -25,6 +25,7 @@
        of issues and comments (e.g. "By <a href="#">@johnsmith</a>"). It will also 
        associate and/or assign these issues and comments with the selected user.
+  .table-holder
    %table.table
      %thead
        %tr

--- a/app/views/import/fogbugz/status.html.haml
+++ b/app/views/import/fogbugz/status.html.haml
@@ -14,7 +14,8 @@
  %p
  = button_tag 'Import all projects', class: 'btn btn-success js-import-all'
-%table.table.import-jobs
+.table-holder
+  %table.table.import-jobs
    %thead
      %tr
        %th From FogBugz

--- a/app/views/import/github/status.html.haml
+++ b/app/views/import/github/status.html.haml
@@ -9,7 +9,8 @@
 %p
  = button_tag 'Import all projects', class: "btn btn-success js-import-all"
-%table.table.import-jobs
+.table-holder
+  %table.table.import-jobs
    %thead
      %tr
        %th From GitHub

--- a/app/views/import/gitlab/status.html.haml
+++ b/app/views/import/gitlab/status.html.haml
@@ -9,7 +9,8 @@
 %p
  = button_tag 'Import all projects', class: "btn btn-success js-import-all"
-%table.table.import-jobs
+.table-holder
+  %table.table.import-jobs
    %thead
      %tr
        %th From GitLab.com

--- a/app/views/import/gitorious/status.html.haml
+++ b/app/views/import/gitorious/status.html.haml
@@ -9,7 +9,8 @@
 %p
  = button_tag 'Import all projects', class: "btn btn-success js-import-all"
-%table.table.import-jobs
+.table-holder
+  %table.table.import-jobs
    %thead
      %tr
        %th From Gitorious.org

--- a/app/views/import/google_code/status.html.haml
+++ b/app/views/import/google_code/status.html.haml
@@ -17,7 +17,8 @@
  - else
    = button_tag 'Import all projects', class: "btn btn-success js-import-all"
-%table.table.import-jobs
+.table-holder
+  %table.table.import-jobs
    %thead
      %tr
        %th From Google Code

--- a/app/views/profiles/_event_table.html.haml
+++ b/app/views/profiles/_event_table.html.haml
-%table.table#audits
+.table-holder
+  %table.table#audits
    %thead
      %tr
        %th Action

--- a/app/views/projects/builds/index.html.haml
+++ b/app/views/projects/builds/index.html.haml
@@ -32,6 +32,7 @@
    %li
      .nothing-here-block No builds to show
  - else
+    .table-holder
      %table.table.builds
        %thead
          %tr

--- a/app/views/projects/ci_web_hooks/index.html.haml
+++ b/app/views/projects/ci_web_hooks/index.html.haml
@@ -20,6 +20,7 @@
 -if @web_hooks.any?
  %h4 Activated web hooks (#{@web_hooks.count})
+  .table-holder
    %table.table
      - @web_hooks.each do |hook|
        %tr

--- a/app/views/projects/commit/ci.html.haml
+++ b/app/views/projects/commit/ci.html.haml
@@ -29,7 +29,8 @@
      - if @ci_commit.builds.running_or_pending.any?
        = link_to "Cancel all", cancel_builds_namespace_project_commit_path(@project.namespace, @project, @commit.sha), class: 'btn btn-xs btn-danger'
-%table.table.builds
+.table-holder
+  %table.table.builds
    %thead
      %tr
        %th Status
@@ -50,6 +51,7 @@
  .gray-content-block.second-block
    Retried builds
+  .table-holder
    %table.table.builds
      %thead
        %tr

--- a/app/views/projects/protected_branches/_branches_list.html.haml
+++ b/app/views/projects/protected_branches/_branches_list.html.haml
 - unless @branches.empty?
  %br
  %h4 Already Protected:
+  .table-holder
    %table.table.protected-branches-list
      %thead
        %tr.no-border

--- a/app/views/projects/runners/show.html.haml
+++ b/app/views/projects/runners/show.html.haml
@@ -9,7 +9,8 @@
        %span.runner-state.runner-state-specific
          Specific
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th Property Name

--- a/app/views/projects/services/index.html.haml
+++ b/app/views/projects/services/index.html.haml
@@ -2,7 +2,8 @@
 %h3.page-title Project services
 %p.light Project services allow you to integrate GitLab with other applications
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th

--- a/app/views/projects/tree/_tree_content.html.haml
+++ b/app/views/projects/tree/_tree_content.html.haml
 %div.tree-content-holder
-  .tree-table-holder
+  .table-holder
    %table.table#tree-slider{class: "table_#{@hex_path} tree-table table-striped" }
      %thead
        %tr

--- a/app/views/projects/triggers/index.html.haml
+++ b/app/views/projects/triggers/index.html.haml
@@ -7,6 +7,7 @@
 %hr.clearfix
 -if @triggers.any?
+  .table-holder
    %table.table
      %thead
        %th Token

--- a/app/views/projects/wikis/history.html.haml
+++ b/app/views/projects/wikis/history.html.haml
@@ -7,7 +7,8 @@
    %span.light History for
    = link_to @page.title, namespace_project_wiki_path(@project.namespace, @project, @page)
-%table.table
+.table-holder
+  %table.table
    %thead
      %tr
        %th Page version