Patch #28510 » 0001-Show-assignee-avatar-in-Roadmap-and-Version.patch
app/helpers/avatars_helper.rb | ||
---|---|---|
24 | 24 |
return '' unless user |
25 | 25 | |
26 | 26 |
options.merge!(:title => l(:field_assigned_to) + ": " + user.name) |
27 |
avatar(user, options).html_safe
|
|
27 |
avatar(user, options) |
|
28 | 28 |
end |
29 | 29 | |
30 | 30 |
def author_avatar(user, options={}) |
31 | 31 |
return '' unless user |
32 | 32 | |
33 | 33 |
options.merge!(:title => l(:field_author) + ": " + user.name) |
34 |
avatar(user, options).html_safe
|
|
34 |
avatar(user, options) |
|
35 | 35 |
end |
36 | 36 | |
37 | 37 |
# Returns the avatar image tag for the given +user+ if avatars are enabled |
app/views/versions/index.html.erb | ||
---|---|---|
28 | 28 |
<table class="list related-issues"> |
29 | 29 |
<caption><%= l(:label_related_issues) %></caption> |
30 | 30 |
<% issues.each do |issue| -%> |
31 |
<tr class="hascontextmenu"> |
|
31 |
<tr class="issue hascontextmenu">
|
|
32 | 32 |
<td class="checkbox"><%= check_box_tag 'ids[]', issue.id, false, :id => nil %></td> |
33 |
<td class="assigned_to"><%= assignee_avatar(issue.assigned_to, :size => 16) %></td> |
|
33 | 34 |
<td class="subject"><%= link_to_issue(issue, :project => (@project != issue.project)) %></td> |
34 | 35 |
<td class="buttons"><%= link_to_context_menu %></td> |
35 | 36 |
</tr> |
app/views/versions/show.html.erb | ||
---|---|---|
44 | 44 |
<%- @issues.each do |issue| -%> |
45 | 45 |
<tr class="issue hascontextmenu"> |
46 | 46 |
<td class="checkbox"><%= check_box_tag 'ids[]', issue.id, false, :id => nil %></td> |
47 |
<td class="assigned_to"><%= assignee_avatar(issue.assigned_to, :size => 16) %></td> |
|
47 | 48 |
<td class="subject"><%= link_to_issue(issue, :project => (@project != issue.project)) %></td> |
48 | 49 |
<td class="buttons"><%= link_to_context_menu %></td> |
49 | 50 |
</tr> |
public/stylesheets/application.css | ||
---|---|---|
582 | 582 | |
583 | 583 |
div#roadmap .related-issues { margin-bottom: 1em; } |
584 | 584 |
div#roadmap .related-issues td.checkbox { display: none; } |
585 |
div#roadmap .related-issues td.assigned_to { width:1px; white-space:nowrap; padding: 0; } |
|
586 |
div#roadmap .related-issues td.assigned_to img { padding-left: 4px; padding-right: 4px;} |
|
585 | 587 |
div#roadmap .wiki h1:first-child { display: none; } |
586 | 588 |
div#roadmap .wiki h1 { font-size: 120%; } |
587 | 589 |
div#roadmap .wiki h2 { font-size: 110%; } |
test/functional/versions_controller_test.rb | ||
---|---|---|
98 | 98 |
end |
99 | 99 |
end |
100 | 100 | |
101 |
def test_index_should_show_issue_assignee |
|
102 |
with_settings :gravatar_enabled => '1' do |
|
103 |
Issue.generate!(:project_id => 3, :fixed_version_id => 4, :assigned_to => User.find_by_login('jsmith')) |
|
104 |
Issue.generate!(:project_id => 3, :fixed_version_id => 4) |
|
105 | ||
106 |
get :index, :params => {:project_id => 3} |
|
107 |
assert_response :success |
|
108 | ||
109 |
assert_select 'table.related-issues' do |
|
110 |
assert_select 'tr.issue', :count => 2 do |
|
111 |
assert_select 'img.gravatar[title=?]', 'Assignee: John Smith', :count => 1 |
|
112 |
end |
|
113 |
end |
|
114 |
end |
|
115 |
end |
|
116 | ||
101 | 117 |
def test_show |
102 | 118 |
get :show, :params => {:id => 2} |
103 | 119 |
assert_response :success |
104 | 120 | |
105 | 121 |
assert_select 'h2', :text => /1.0/ |
106 | 122 |
assert_select 'span[class=?]', 'badge badge-status-locked', :text => 'locked' |
123 | ||
124 |
# no issue avatar when gravatar is disabled |
|
125 |
assert_select 'img.gravatar', :count => 0 |
|
126 |
end |
|
127 | ||
128 |
def test_show_should_show_issue_assignee |
|
129 |
with_settings :gravatar_enabled => '1' do |
|
130 |
get :show, :params => {:id => 2} |
|
131 |
assert_response :success |
|
132 | ||
133 |
assert_select 'table.related-issues' do |
|
134 |
assert_select 'tr.issue td.assigned_to', :count => 2 do |
|
135 |
assert_select 'img.gravatar[title=?]', 'Assignee: Dave Lopper', :count => 1 |
|
136 |
end |
|
137 |
end |
|
138 |
end |
|
107 | 139 |
end |
108 | 140 | |
109 | 141 |
def test_show_issue_calculations_should_take_into_account_only_visible_issues |
- « Previous
- 1
- …
- 10
- 11
- 12
- Next »