Merge branch 'mk/geo-deprecate-legacy-storage-ce' into 'master'
Backport persistent user callout flash message to CE See merge request gitlab-org/gitlab-ce!24166
This commit is contained in:
commit
75e6561684
3 changed files with 24 additions and 0 deletions
|
@ -13,6 +13,10 @@ module UserCalloutsHelper
|
|||
!user_dismissed?(GCP_SIGNUP_OFFER)
|
||||
end
|
||||
|
||||
def render_flash_user_callout(flash_type, message, feature_name)
|
||||
render 'shared/flash_user_callout', flash_type: flash_type, message: message, feature_name: feature_name
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
def user_dismissed?(feature_name)
|
||||
|
|
11
app/views/shared/_flash_user_callout.html.haml
Normal file
11
app/views/shared/_flash_user_callout.html.haml
Normal file
|
@ -0,0 +1,11 @@
|
|||
- callout_data = { uid: "callout_feature_#{feature_name}_dismissed", feature_id: feature_name, dismiss_endpoint: user_callouts_path }
|
||||
- extra_flash_class = local_assigns.fetch(:extra_flash_class, nil)
|
||||
|
||||
.flash-container.flash-container-page.user-callout{ data: callout_data }
|
||||
-# We currently only support `alert`, `warning`, `notice`, `success`
|
||||
%div{ class: "flash-#{flash_type}" }
|
||||
%div{ class: "#{(container_class unless fluid_layout)} #{(extra_flash_class unless @no_container)} #{@content_class}" }
|
||||
%span= message
|
||||
%button.btn.btn-default.close.js-close{ type: 'button',
|
||||
'aria-label' => _('Dismiss') }
|
||||
= sprite_icon('close', css_class: 'dismiss-icon')
|
|
@ -44,4 +44,13 @@ describe UserCalloutsHelper do
|
|||
it { is_expected.to be false }
|
||||
end
|
||||
end
|
||||
|
||||
describe '.render_flash_user_callout' do
|
||||
it 'renders the flash_user_callout partial' do
|
||||
expect(helper).to receive(:render)
|
||||
.with(/flash_user_callout/, flash_type: :warning, message: 'foo', feature_name: 'bar')
|
||||
|
||||
helper.render_flash_user_callout(:warning, 'foo', 'bar')
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Reference in a new issue