summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOswaldo Ferreira <oswaldo@gitlab.com>2018-06-27 17:48:05 -0300
committerOswaldo Ferreira <oswaldo@gitlab.com>2018-06-27 17:48:05 -0300
commitaf24e871898a4ecebd00266d49310dcf9f4d7b5d (patch)
tree86d09e97aa31dead8338796611f9872b186282b4
parent07de43a7e07cefcbb60e7ed84c684f478eae8b93 (diff)
downloadgitlab-ce-18141-osw-use-monospaced-font-on-diffs-commit-ref.tar.gz
Use monospaced font for MR diff commit link on GFM18141-osw-use-monospaced-font-on-diffs-commit-ref
-rw-r--r--changelogs/unreleased/18141-osw-use-monospaced-font-on-diffs-commit-ref.yml5
-rw-r--r--lib/banzai/filter/merge_request_reference_filter.rb5
-rw-r--r--lib/banzai/filter/reference_filter.rb8
-rw-r--r--spec/lib/banzai/filter/merge_request_reference_filter_spec.rb7
4 files changed, 22 insertions, 3 deletions
diff --git a/changelogs/unreleased/18141-osw-use-monospaced-font-on-diffs-commit-ref.yml b/changelogs/unreleased/18141-osw-use-monospaced-font-on-diffs-commit-ref.yml
new file mode 100644
index 00000000000..43ff880a8cb
--- /dev/null
+++ b/changelogs/unreleased/18141-osw-use-monospaced-font-on-diffs-commit-ref.yml
@@ -0,0 +1,5 @@
+---
+title: Use monospaced font for MR diff commit link ref on GFM
+merge_request:
+author:
+type: other
diff --git a/lib/banzai/filter/merge_request_reference_filter.rb b/lib/banzai/filter/merge_request_reference_filter.rb
index 5cbdb01c130..24a0c48b850 100644
--- a/lib/banzai/filter/merge_request_reference_filter.rb
+++ b/lib/banzai/filter/merge_request_reference_filter.rb
@@ -25,7 +25,10 @@ module Banzai
extras = super
if commit_ref = object_link_commit_ref(object, matches)
- return extras.unshift(commit_ref)
+ klass = reference_class(:commit, tooltip: false)
+ commit_ref_tag = %(<span class="#{klass}">#{commit_ref}</a>)
+
+ return extras.unshift(commit_ref_tag)
end
path = matches[:path] if matches.names.include?("path")
diff --git a/lib/banzai/filter/reference_filter.rb b/lib/banzai/filter/reference_filter.rb
index 2f023f4f242..2411dd2cdfc 100644
--- a/lib/banzai/filter/reference_filter.rb
+++ b/lib/banzai/filter/reference_filter.rb
@@ -65,8 +65,12 @@ module Banzai
context[:skip_project_check]
end
- def reference_class(type)
- "gfm gfm-#{type} has-tooltip"
+ def reference_class(type, tooltip: true)
+ gfm_klass = "gfm gfm-#{type}"
+
+ return gfm_klass unless tooltip
+
+ "#{gfm_klass} has-tooltip"
end
# Ensure that a :project key exists in context
diff --git a/spec/lib/banzai/filter/merge_request_reference_filter_spec.rb b/spec/lib/banzai/filter/merge_request_reference_filter_spec.rb
index a1dd72c498f..55c41e55437 100644
--- a/spec/lib/banzai/filter/merge_request_reference_filter_spec.rb
+++ b/spec/lib/banzai/filter/merge_request_reference_filter_spec.rb
@@ -210,6 +210,13 @@ describe Banzai::Filter::MergeRequestReferenceFilter do
.to eq reference
end
+ it 'commit ref tag is valid' do
+ doc = reference_filter("See #{reference}")
+ commit_ref_tag = doc.css('a').first.css('span.gfm.gfm-commit')
+
+ expect(commit_ref_tag.text).to eq(commit.short_id)
+ end
+
it 'has valid text' do
doc = reference_filter("See #{reference}")