diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2016-02-26 10:52:00 +0100 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2016-03-03 10:41:11 +0100 |
commit | 62dcdef969fe29f43abfbe2653e73ab09b2d1bf6 (patch) | |
tree | d8c1b1a2679335c923c1c8b676aba53bdd3640a4 | |
parent | 7183a7da6dbe5d2faf2ae03dfe4652ca6ed6dfde (diff) | |
download | gitlab-ce-62dcdef969fe29f43abfbe2653e73ab09b2d1bf6.tar.gz |
Improve cross project label look and feel
-rw-r--r-- | app/helpers/labels_helper.rb | 11 | ||||
-rw-r--r-- | spec/lib/banzai/filter/label_reference_filter_spec.rb | 10 |
2 files changed, 14 insertions, 7 deletions
diff --git a/app/helpers/labels_helper.rb b/app/helpers/labels_helper.rb index 122f50081e4..37fc1dab876 100644 --- a/app/helpers/labels_helper.rb +++ b/app/helpers/labels_helper.rb @@ -50,22 +50,23 @@ module LabelsHelper @project.labels.pluck(:title) end - def render_colored_label(label, label_prefix = '') + def render_colored_label(label, label_suffix = '') label_color = label.color || Label::DEFAULT_COLOR text_color = text_color_for_bg(label_color) # Intentionally not using content_tag here so that this method can be called # by LabelReferenceFilter span = %(<span class="label color-label") + - %( style="background-color: #{label_color}; color: #{text_color}">) + - label_prefix + escape_once(label.name) + '</span>' + %(style="background-color: #{label_color}; color: #{text_color}">) + + %(#{escape_once(label.name)}#{label_suffix}</span>) span.html_safe end def render_colored_cross_project_label(label) - label_prefix = "#{label.project.path_with_namespace} » " - render_colored_label(label, label_prefix) + label_suffix = label.project.name + label_suffix = " <i>« #{label_suffix}</i>" + render_colored_label(label, label_suffix) end def suggested_colors diff --git a/spec/lib/banzai/filter/label_reference_filter_spec.rb b/spec/lib/banzai/filter/label_reference_filter_spec.rb index 96a403f2694..8786095fbe4 100644 --- a/spec/lib/banzai/filter/label_reference_filter_spec.rb +++ b/spec/lib/banzai/filter/label_reference_filter_spec.rb @@ -178,7 +178,8 @@ describe Banzai::Filter::LabelReferenceFilter, lib: true do end describe 'cross project label references' do - let(:another_project) { create(:empty_project, :public) } + let(:project_name) { 'Some project' } + let(:another_project) { create(:empty_project, :public, name: project_name) } let(:label) { create(:label, project: another_project, color: '#00ff00') } let(:reference) { label.to_reference(project) } @@ -192,7 +193,12 @@ describe Banzai::Filter::LabelReferenceFilter, lib: true do end it 'has valid color' do - expect(result.css('a span').first.attr('style')).to match /background-color: #00ff00/ + expect(result.css('a span').first.attr('style')) + .to match /background-color: #00ff00/ + end + + it 'contains cross project content' do + expect(result.css('a').first.text).to eq "#{label.name} « #{project_name}" end end end |