Feature #33572 » 0001-Update-tooltip-title-of-next-prev-issue-link.patch
| app/controllers/issues_controller.rb | ||
|---|---|---|
| 470 | 470 | |
| 471 | 471 |
def retrieve_previous_and_next_issue_ids |
| 472 | 472 |
if params[:prev_issue_id].present? || params[:next_issue_id].present? |
| 473 |
@prev_issue_id = params[:prev_issue_id].presence.try(:to_i)
|
|
| 474 |
@next_issue_id = params[:next_issue_id].presence.try(:to_i)
|
|
| 473 |
@prev_issue = Issue.find_by(id: params[:prev_issue_id].presence.try(:to_i))
|
|
| 474 |
@next_issue = Issue.find_by(id: params[:next_issue_id].presence.try(:to_i))
|
|
| 475 | 475 |
@issue_position = params[:issue_position].presence.try(:to_i) |
| 476 | 476 |
@issue_count = params[:issue_count].presence.try(:to_i) |
| 477 | 477 |
else |
| ... | ... | |
| 485 | 485 |
@issue_position = idx + 1 |
| 486 | 486 |
@issue_count = issue_ids.size |
| 487 | 487 |
end |
| 488 |
@prev_issue_id = issue_ids[idx - 1] if idx > 0
|
|
| 489 |
@next_issue_id = issue_ids[idx + 1] if idx < (issue_ids.size - 1)
|
|
| 488 |
@prev_issue = Issue.find_by(id: issue_ids[idx - 1]) if idx > 0
|
|
| 489 |
@next_issue = Issue.find_by(id: issue_ids[idx + 1]) if idx < (issue_ids.size - 1)
|
|
| 490 | 490 |
end |
| 491 | 491 |
query_params = @query.as_params |
| 492 | 492 |
if @issue_position |
| app/views/issues/_edit.html.erb | ||
|---|---|---|
| 77 | 77 |
l(:button_cancel), issue_path(id: @issue.id), |
| 78 | 78 |
:onclick => params[:action] == 'show' ? "$('#update').hide(); return false;" : ''
|
| 79 | 79 |
) %> |
| 80 |
<%= hidden_field_tag 'prev_issue_id', @prev_issue_id if @prev_issue_id %>
|
|
| 81 |
<%= hidden_field_tag 'next_issue_id', @next_issue_id if @next_issue_id %>
|
|
| 80 |
<%= hidden_field_tag 'prev_issue_id', @prev_issue.id if @prev_issue %>
|
|
| 81 |
<%= hidden_field_tag 'next_issue_id', @next_issue.id if @next_issue %>
|
|
| 82 | 82 |
<%= hidden_field_tag 'issue_position', @issue_position if @issue_position %> |
| 83 | 83 |
<%= hidden_field_tag 'issue_count', @issue_count if @issue_count %> |
| 84 | 84 |
<% end %> |
| app/views/issues/show.html.erb | ||
|---|---|---|
| 4 | 4 |
<%= issue_status_type_badge(@issue.status) %> |
| 5 | 5 | |
| 6 | 6 |
<div class="<%= @issue.css_classes %> details"> |
| 7 |
<% if @prev_issue_id || @next_issue_id %>
|
|
| 7 |
<% if @prev_issue || @next_issue %>
|
|
| 8 | 8 |
<div class="next-prev-links contextual"> |
| 9 |
<%= link_to_if @prev_issue_id,
|
|
| 9 |
<%= link_to_if @prev_issue, |
|
| 10 | 10 |
"\xc2\xab #{l(:label_previous)}",
|
| 11 |
(@prev_issue_id ? issue_path(@prev_issue_id) : nil),
|
|
| 12 |
:title => "##{@prev_issue_id}",
|
|
| 11 |
(@prev_issue&.id ? issue_path(@prev_issue.id) : nil),
|
|
| 12 |
:title => @prev_issue.to_s,
|
|
| 13 | 13 |
:accesskey => accesskey(:previous) %> | |
| 14 | 14 |
<% if @issue_position && @issue_count %> |
| 15 | 15 |
<span class="position"> |
| ... | ... | |
| 18 | 18 |
@query_path %> |
| 19 | 19 |
</span> | |
| 20 | 20 |
<% end %> |
| 21 |
<%= link_to_if @next_issue_id,
|
|
| 21 |
<%= link_to_if @next_issue, |
|
| 22 | 22 |
"#{l(:label_next)} \xc2\xbb",
|
| 23 |
(@next_issue_id ? issue_path(@next_issue_id) : nil),
|
|
| 24 |
:title => "##{@next_issue_id}",
|
|
| 23 |
(@next_issue&.id ? issue_path(@next_issue.id) : nil),
|
|
| 24 |
:title => @next_issue.to_s,
|
|
| 25 | 25 |
:accesskey => accesskey(:next) %> |
| 26 | 26 |
</div> |
| 27 | 27 |
<% end %> |