diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2012-04-15 21:01:35 +0300 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2012-04-15 21:01:35 +0300 |
commit | a7ed8276d7d9448f454f7bbfe8b4a3fb305ae446 (patch) | |
tree | b73b22bc98ae7844183618e43a2221c5a6d659fe /app | |
parent | 86d7b4f152b806b0f53c801a88b7fba016943a94 (diff) | |
parent | ad25dac8b6e7884a333591a6337bd38f3eea0740 (diff) | |
download | gitlab-ce-a7ed8276d7d9448f454f7bbfe8b4a3fb305ae446.tar.gz |
Merge branch 'nicer_commit_headers' of https://github.com/arthurschreiber/gitlabhq into arthurschreiber-nicer_commit_headers
Diffstat (limited to 'app')
-rw-r--r-- | app/assets/stylesheets/main.scss | 7 | ||||
-rw-r--r-- | app/assets/stylesheets/sections/commits.scss | 58 | ||||
-rw-r--r-- | app/models/commit.rb | 35 | ||||
-rw-r--r-- | app/views/commits/show.html.haml | 49 |
4 files changed, 126 insertions, 23 deletions
diff --git a/app/assets/stylesheets/main.scss b/app/assets/stylesheets/main.scss index 141f2acbdf6..a6124170f44 100644 --- a/app/assets/stylesheets/main.scss +++ b/app/assets/stylesheets/main.scss @@ -94,7 +94,12 @@ $hover: #FDF5D9; @import "common.scss"; /** - * This scss file redefine chozen selectbox styles for + * Styles related to displaying commits + */ +@import "sections/commits.scss"; + +/** + * This scss file redefine chozen selectbox styles for * project Branch/Tag select element */ @import "ref_select.scss"; diff --git a/app/assets/stylesheets/sections/commits.scss b/app/assets/stylesheets/sections/commits.scss new file mode 100644 index 00000000000..85e7315ec89 --- /dev/null +++ b/app/assets/stylesheets/sections/commits.scss @@ -0,0 +1,58 @@ +.commit-head { + @extend .main_box; + + padding: 14px; + padding-bottom: 8px; + line-height: 24px; + + .browse-button { + @extend .btn; + @extend .btn-small; + float: right; + } + + .commit-title { + line-height: 26px; + } + + .commit-description { + font-size: 14px; + padding: 0px; + padding-bottom: 4px; + border: none; + background-color: white; + } + + .commit-info { + @extend .middle_box_content; + @extend .clearfix; + + padding-bottom: 8px; + padding-top: 8px; + + margin-left: -14px; + margin-right: -14px; + margin-bottom: -8px; + + .author .name, .committer .name { + font-weight: bold; + } + } + + .sha-block { + float: right; + margin-left: 10px + } + + &.merge-commit .sha-block { + clear: right; + } + + .committer { + padding-left: 32px; + } + + .avatar { + margin-right: 4px; + } +}
\ No newline at end of file diff --git a/app/models/commit.rb b/app/models/commit.rb index 76064b05670..343235e58af 100644 --- a/app/models/commit.rb +++ b/app/models/commit.rb @@ -106,6 +106,41 @@ class Commit utf8 author.name end + # Returns the commits title. + # + # Usually, the commit title is the first line of the commit message. + # In case this first line is longer than 80 characters, it is cut off + # after 70 characters and ellipses (`&hellp;`) are appended. + # + # @todo This might be better placed in a view helper. + def title + title_end = safe_message.index(/\n/) + if (!title_end && safe_message.length > 80) || (title_end && title_end > 80) + safe_message[0..69] << "…".html_safe + else + safe_message.split(/\n/, 2).first + end + end + + # Returns the commits description + # + # cut off, ellipses (`&hellp;`) are prepended to the commit message. + # + # @todo This might be better placed in a view helper. + def description + title_end = safe_message.index(/\n/) + if (!title_end && safe_message.length > 80) || (title_end && title_end > 80) + "…".html_safe << safe_message[70..-1] + else + safe_message.split(/\n/, 2)[1].try(:chomp) + end + end + + # Was this commit committed by a different person than the original author? + def different_committer? + author_name != committer_name || author_email != committer_email + end + def committer_name utf8 committer.name end diff --git a/app/views/commits/show.html.haml b/app/views/commits/show.html.haml index dc63e0e821c..eae8c804c81 100644 --- a/app/views/commits/show.html.haml +++ b/app/views/commits/show.html.haml @@ -1,26 +1,31 @@ -.main_box - .top_box_content - .right - - unless @notes_count.zero? - %span.btn.small.disabled.padded= pluralize @notes_count, 'note' +.commit-head{class: @commit.parents.count > 1 ? "merge-commit" : ""} + = link_to "Browse Code »", tree_project_ref_path(@project, @commit.id), :class => "browse-button" + %h3.commit-title + = commit_msg_with_link_to_issues(@project, @commit.title) + - if @commit.description.present? + %pre.commit-description + = commit_msg_with_link_to_issues(@project, @commit.description) + .commit-info + %span.sha-block + commit + %code= @commit.id + %span.sha-block + = pluralize(@commit.parents.count, "parent") + - @commit.parents.each do |parent| + %code= link_to parent.id, project_commit_path(@project, parent) + .author + = image_tag gravatar_icon(@commit.author_email, 24), :class => "avatar", :height => 24, :width => 24 + %span.name= @commit.author_name + authored + %time{title: @commit.authored_date.stamp("Aug 21, 2011 9:23pm")} + #{time_ago_in_words(@commit.authored_date)} ago + - if @commit.different_committer? + .committer + %span.name= @commit.committer_name + committed + %time{title: @commit.committed_date.stamp("Aug 21, 2011 9:23pm")} + #{time_ago_in_words(@commit.committed_date)} ago - = link_to tree_project_ref_path(@project, @commit.id), :class => "btn small" do - Browse Code » - = image_tag gravatar_icon(@commit.author_email), :class => "avatar" - %code= @commit.id.to_s - %h5 - = @commit.author_name - %small= @commit.created_at.stamp("Aug 21, 2011 9:23pm") - - if @commit.author_name != @commit.committer_name or @commit.author_email != @commit.committer_email or @commit.authored_date != @commit.committed_date - – - %cite committed by - = @commit.committer_name - %small= @commit.committed_date.stamp("Aug 21, 2011 9:23pm") - - .middle_box_content - %pre.commit_message - = commit_msg_with_link_to_issues(@project, @commit.safe_message) -%br = render "commits/diffs", :diffs => @commit.diffs = render "notes/notes", :tid => @commit.id, :tt => "commit" = render "notes/per_line_form" |