Commit 3704d85bc49e76891f37f3b14d6fddb499921ed8
1 parent
ae1a3148
Exists in
spb-stable
and in
3 other branches
Remove periods from merge request
The "Merged by" and "Closed by" statements end with an extra space followed by a period. This doesn't seem necessary to me and not having the extra space followed by a period looks better to me. Also, the top of the merge request doesn't have the sentence ending with a period.
Showing
1 changed file
with
2 additions
and
2 deletions
Show diff stats
app/views/projects/merge_requests/show/_mr_box.html.haml
| @@ -29,13 +29,13 @@ | @@ -29,13 +29,13 @@ | ||
| 29 | %span | 29 | %span |
| 30 | %i.icon-remove | 30 | %i.icon-remove |
| 31 | Closed by #{link_to_member(@project, @merge_request.closed_event.author)} | 31 | Closed by #{link_to_member(@project, @merge_request.closed_event.author)} |
| 32 | - #{time_ago_with_tooltip(@merge_request.closed_event.created_at)}. | 32 | + #{time_ago_with_tooltip(@merge_request.closed_event.created_at)} |
| 33 | - if @merge_request.merged? | 33 | - if @merge_request.merged? |
| 34 | .alert.alert-info | 34 | .alert.alert-info |
| 35 | %span | 35 | %span |
| 36 | %i.icon-ok | 36 | %i.icon-ok |
| 37 | Merged by #{link_to_member(@project, @merge_request.merge_event.author)} | 37 | Merged by #{link_to_member(@project, @merge_request.merge_event.author)} |
| 38 | - #{time_ago_with_tooltip(@merge_request.merge_event.created_at)}. | 38 | + #{time_ago_with_tooltip(@merge_request.merge_event.created_at)} |
| 39 | - if !@closes_issues.empty? && @merge_request.open? | 39 | - if !@closes_issues.empty? && @merge_request.open? |
| 40 | .alert.alert-info.alert-info | 40 | .alert.alert-info.alert-info |
| 41 | %span | 41 | %span |