diff options
author | Connor Shea <connor.james.shea@gmail.com> | 2016-05-24 15:24:11 -0600 |
---|---|---|
committer | Connor Shea <connor.james.shea@gmail.com> | 2016-05-24 15:24:11 -0600 |
commit | 77a1565c806fece3a1d664c2c069f38dcb371c5c (patch) | |
tree | 92f9b23d9722a30bc8c957f74af92427aaffc340 | |
parent | a9b431c5b91af5c082b22579a04c5516576f1819 (diff) | |
download | gitlab-ce-link-to-in-issues-index.tar.gz |
Replace `link_to_gfm` with `link_to` in merge request and issue titles.link-to-in-issues-index
Previously if an issue had an emoji for its name it would be unclickable. Similarly, links were rendered inline if the title was something like “Fixes a bug with https://google.com”. This confused some users, so it’s removed.
On the issue/MR page itself, the formatting is preserved. This only effects index pages.
This is also ever so slightly more performant :)
This fixes #17614 and #17230.
-rw-r--r-- | app/views/projects/issues/_issue.html.haml | 2 | ||||
-rw-r--r-- | app/views/projects/merge_requests/_merge_request.html.haml | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/app/views/projects/issues/_issue.html.haml b/app/views/projects/issues/_issue.html.haml index 5cf70ea3bb7..78f64150601 100644 --- a/app/views/projects/issues/_issue.html.haml +++ b/app/views/projects/issues/_issue.html.haml @@ -6,7 +6,7 @@ .issue-title.title %span.issue-title-text = confidential_icon(issue) - = link_to_gfm issue.title, issue_path(issue) + = link_to issue.title, issue_path(issue) %ul.controls - if issue.closed? %li diff --git a/app/views/projects/merge_requests/_merge_request.html.haml b/app/views/projects/merge_requests/_merge_request.html.haml index 2c54171c6bd..c02f94490a0 100644 --- a/app/views/projects/merge_requests/_merge_request.html.haml +++ b/app/views/projects/merge_requests/_merge_request.html.haml @@ -1,7 +1,7 @@ %li{ class: mr_css_classes(merge_request) } .merge-request-title.title %span.merge-request-title-text - = link_to_gfm merge_request.title, merge_request_path(merge_request) + = link_to merge_request.title, merge_request_path(merge_request) %ul.controls - if merge_request.merged? %li |