Improve Commit#show page UI

app/assets/stylesheets/sections/commits.scss

@@ -440,17 +440,24 @@
   margin-top: 10px;
 }
 
-.ui-box.commit-box {
-  margin-top: 0;
+.commit-box {
+  margin: 10px 0;
+  border-top: 1px solid #ddd;
+  border-bottom: 1px solid #ddd;
+  padding: 20px 0;
 
-  .commit-committer-link,
-  .commit-author-link {
-    color: #333;
+  .commit-title {
+    margin: 0;
+    font-size: 20px;
     font-weight: bold;
-    text-shadow: 0 1px 1px #FFF;
+  }
+
+  .commit-description {
+    margin-top: 15px;
   }
 }
 
+
 .commit-stat-summary {
   color: #666;
   line-height: 2;
@@ -477,3 +484,15 @@ li.commit {
 .commit-breadcrumb {
   padding: 0;
 }
+
+.commit-info-row {
+  margin-bottom: 10px;
+  .avatar {
+    @extend .avatar-inline;
+  }
+  .commit-committer-link,
+  .commit-author-link {
+    color: #444;
+    font-weight: bold;
+  }
+}

app/views/projects/commit/_commit_box.html.haml

-.ui-box.ui-box-show.commit-box
-  .ui-box-head
-    .pull-right
+.pull-right
+  %div
     - if @notes_count > 0
       %span.btn.disabled.grouped
         %i.icon-comment
         = @notes_count
-      .left.btn-group
+    .pull-left.btn-group
       %a.btn.grouped.dropdown-toggle{ data: {toggle: :dropdown} }
         %i.icon-download-alt
         Download as
@@ -15,36 +14,34 @@
         %li= link_to "Plain Diff",    project_commit_path(@project, @commit, format: :diff)
     = link_to project_tree_path(@project, @commit), class: "btn btn-primary grouped" do
       %span Browse Code »
-    %h3.commit-title.page-title
-      = gfm escape_once(@commit.title)
-    - if @commit.description.present?
-      %pre.commit-description
-        = gfm escape_once(@commit.description)
-  .ui-box-body
-    .row
-      .span5
-        .author
-          = commit_author_link(@commit, avatar: true, size: 32)
-          authored
+  %div
+
+%p
+  %span.light Commit
+  = link_to @commit.id, project_commit_path(@project, @commit)
+.commit-info-row
+  %span.light Authored by
+  %strong
+    = commit_author_link(@commit, avatar: true, size: 24)
   %time{title: @commit.authored_date.stamp("Aug 21, 2011 9:23pm")}
     #{time_ago_in_words(@commit.authored_date)} ago
-        - if @commit.different_committer?
-          .committer
-            →
-            = commit_committer_link(@commit)
-            committed
+
+- if @commit.different_committer?
+  .commit-info-row
+    %span.light Committed by
+    %strong
+      = commit_committer_link(@commit, avatar: true, size: 24)
     %time{title: @commit.committed_date.stamp("Aug 21, 2011 9:23pm")}
       #{time_ago_in_words(@commit.committed_date)} ago
-      .span6.pull-right
-        .pull-right
-          .sha-block
-            %span.cgray commit
-            %span.label_commit= @commit.id
-        .clearfix
-        .pull-right
-          .sha-block
+
+.commit-info-row
   %span.cgray= pluralize(@commit.parents.count, "parent")
   - @commit.parents.each do |parent|
     = link_to parent.id[0...10], project_commit_path(@project, parent)
 
-
+.commit-box
+  %h3.commit-title
+    = gfm escape_once(@commit.title)
+  - if @commit.description.present?
+    %pre.commit-description
+      = gfm escape_once(@commit.description)