Update author info in tree view

Riyad Preukschas
1 parent aef2031a
Showing 4 changed files with 23 additions and 13 deletions Show diff stats
app/assets/stylesheets/sections/tree.scss
app/views/refs/logs_tree.js.haml
app/views/tree/_tree_commit.html.haml
app/views/tree/_tree_commit_column.html.haml
@@ -52,14 +52,26 @@
     }
   }
-  .tree-commit-link {
-    color:#333;
+  .tree_author {
+    padding-right: 8px;
+
+    img.avatar {
+      border: 0 none;
+      float: none;
+      margin-right: 0;
+      padding: 0;
+      width: 16px;
+    }
   }
-  a.tree-commit-link {
-    color: #666;
-    &:hover {
-      text-decoration: underline;
+  .tree_commit {
+    color: gray;
+
+    .tree-commit-link {
+      color: #444;
+      &:hover {
+        text-decoration: underline;
+      }
     }
   }
 }
 - @logs.each do |content_data| 
   - file_name = content_data[:file_name]
-  - content_commit = content_data[:commit]
-  - tm = @project.team_member_by_name_or_email(content_commit.author_email, content_commit.author_name)
+  - commit = content_data[:commit]
   :plain
     var row = $("table.table_#{@hex_path} tr.file_#{hexdigest(file_name)}");
-    row.find("td.tree_time_ago").html('#{escape_javascript(time_ago_in_words(content_commit.committed_date))} ago');
-    row.find("td.tree_commit").html('#{escape_javascript(render("tree/tree_commit", tm: tm, content_commit: content_commit))}');
+    row.find("td.tree_time_ago").html('#{escape_javascript time_ago_in_words(commit.committed_date)} ago');
+    row.find("td.tree_commit").html('#{escape_javascript render("tree/tree_commit_column", commit: commit)}');
@@ -1,3 +0,0 @@
-- if tm
-  = link_to "[#{tm.user_name}]", project_team_member_path(@project, tm)
-= link_to_gfm truncate(content_commit.title, length: tm ? 30 : 50), project_commit_path(@project, content_commit.id), class: "tree-commit-link"
@@ -0,0 +1,2 @@
+%span.tree_author= commit.author_link avatar: true
+= link_to_gfm truncate(commit.title, length: 80), project_commit_path(@project, commit.id), class: "tree-commit-link"
	@@ -52,14 +52,26 @@		@@ -52,14 +52,26 @@
52	}	52	}
53	}	53	}
54		54
55	- .tree-commit-link {
56	- color:#333;	55	+ .tree_author {
		56	+ padding-right: 8px;
		57	+
		58	+ img.avatar {
		59	+ border: 0 none;
		60	+ float: none;
		61	+ margin-right: 0;
		62	+ padding: 0;
		63	+ width: 16px;
		64	+ }
57	}	65	}
58		66
59	- a.tree-commit-link {
60	- color: #666;
61	- &:hover {
62	- text-decoration: underline;	67	+ .tree_commit {
		68	+ color: gray;
		69	+
		70	+ .tree-commit-link {
		71	+ color: #444;
		72	+ &:hover {
		73	+ text-decoration: underline;
		74	+ }
63	}	75	}
64	}	76	}
65	}	77	}
	@@ -1,3 +0,0 @@	@@ -1,3 +0,0 @@
1	-- if tm
2	- = link_to "[#{tm.user_name}]", project_team_member_path(@project, tm)
3	-= link_to_gfm truncate(content_commit.title, length: tm ? 30 : 50), project_commit_path(@project, content_commit.id), class: "tree-commit-link"
@@ -0,0 +1,2 @@		@@ -0,0 +1,2 @@
	1	+%span.tree_author= commit.author_link avatar: true
	2	+= link_to_gfm truncate(commit.title, length: 80), project_commit_path(@project, commit.id), class: "tree-commit-link"