diff --git a/app/assets/images/user-signup-ial2.svg b/app/assets/images/user-signup-ial2.svg deleted file mode 100644 index a622cb916e7..00000000000 --- a/app/assets/images/user-signup-ial2.svg +++ /dev/null @@ -1 +0,0 @@ - \ No newline at end of file diff --git a/app/assets/images/user-signup-ial1.svg b/app/assets/images/user-signup.svg similarity index 100% rename from app/assets/images/user-signup-ial1.svg rename to app/assets/images/user-signup.svg diff --git a/app/presenters/completions_presenter.rb b/app/presenters/completions_presenter.rb index 52a17f5c60f..e83a3bb37f3 100644 --- a/app/presenters/completions_presenter.rb +++ b/app/presenters/completions_presenter.rb @@ -102,18 +102,6 @@ def intro end end - def image_name - if ial2_requested? - 'user-signup-ial2.svg' - else - 'user-signup-ial1.svg' - end - end - - def image_alt - I18n.t('sign_up.completed.smiling_image_alt') - end - def pii displayable_attribute_keys.index_with do |attribute_name| displayable_pii[attribute_name] diff --git a/app/views/mfa_confirmation/show.html.erb b/app/views/mfa_confirmation/show.html.erb index 806ece8ff5e..06e8bca9cb8 100644 --- a/app/views/mfa_confirmation/show.html.erb +++ b/app/views/mfa_confirmation/show.html.erb @@ -1,6 +1,6 @@ <% self.title = @content.heading %> -<%= image_tag asset_url('user-signup-ial1.svg'), width: 107, height: 119, alt: '', class: 'margin-bottom-4', aria: { hidden: true } %> +<%= image_tag asset_url('user-signup.svg'), width: 107, height: 119, alt: '', class: 'margin-bottom-4', aria: { hidden: true } %> <%= render PageHeadingComponent.new.with_content(@content.heading) %> diff --git a/app/views/sign_up/completions/show.html.erb b/app/views/sign_up/completions/show.html.erb index 0ddd6e6dfa2..6dbe1572268 100644 --- a/app/views/sign_up/completions/show.html.erb +++ b/app/views/sign_up/completions/show.html.erb @@ -1,6 +1,13 @@ <% self.title = @presenter.heading %> +
- <%= image_tag asset_url(@presenter.image_name), width: 140, alt: @presenter.image_alt, class: 'margin-bottom-2' %> + <%= image_tag( + asset_url('user-signup.svg'), + width: 140, + height: 156, + alt: t('sign_up.completed.smiling_image_alt'), + class: 'margin-bottom-2', + ) %>
<%= render PageHeadingComponent.new(class: 'text-center') do %> diff --git a/app/views/users/backup_code_setup/reminder.html.erb b/app/views/users/backup_code_setup/reminder.html.erb index c36c6d72b1d..770ab54031e 100644 --- a/app/views/users/backup_code_setup/reminder.html.erb +++ b/app/views/users/backup_code_setup/reminder.html.erb @@ -1,6 +1,6 @@ <% self.title = t('forms.backup_code.title') %> -<%= image_tag asset_url('user-signup-ial1.svg'), width: 107, height: 119, alt: '', class: 'margin-bottom-4', aria: { hidden: true } %> +<%= image_tag asset_url('user-signup.svg'), width: 107, height: 119, alt: '', class: 'margin-bottom-4', aria: { hidden: true } %> <%= render PageHeadingComponent.new.with_content(t('forms.backup_code_reminder.heading')) %> diff --git a/spec/presenters/completions_presenter_spec.rb b/spec/presenters/completions_presenter_spec.rb index c9bf04488a5..102087eaa70 100644 --- a/spec/presenters/completions_presenter_spec.rb +++ b/spec/presenters/completions_presenter_spec.rb @@ -146,30 +146,6 @@ end end - describe '#image_name' do - context 'ial2 sign in' do - let(:ial2_requested) { true } - - it 'renders the ial2 image' do - expect(presenter.image_name).to eq('user-signup-ial2.svg') - end - end - - context 'ial1 sign in' do - let(:ial2_requested) { false } - - it 'renders the ial1 image' do - expect(presenter.image_name).to eq('user-signup-ial1.svg') - end - end - end - - describe '#image_alt' do - it 'returns image alt test' do - expect(presenter.image_alt).to eq(I18n.t('sign_up.completed.smiling_image_alt')) - end - end - describe '#intro' do describe 'ial1' do context 'consent has expired since the last sign in' do