diff --git a/app/assets/images/mfa-options/security_key.gif b/app/assets/images/mfa-options/security_key.gif deleted file mode 100644 index ed40944237a..00000000000 Binary files a/app/assets/images/mfa-options/security_key.gif and /dev/null differ diff --git a/app/assets/images/mfa-options/security_key.svg b/app/assets/images/mfa-options/security_key.svg new file mode 100644 index 00000000000..25e8722f75d --- /dev/null +++ b/app/assets/images/mfa-options/security_key.svg @@ -0,0 +1 @@ + \ No newline at end of file diff --git a/app/assets/images/mfa-options/security_key_mobile.gif b/app/assets/images/mfa-options/security_key_mobile.gif deleted file mode 100644 index f14234f9af8..00000000000 Binary files a/app/assets/images/mfa-options/security_key_mobile.gif and /dev/null differ diff --git a/app/assets/images/mfa-options/security_key_mobile.svg b/app/assets/images/mfa-options/security_key_mobile.svg new file mode 100644 index 00000000000..7433b38df38 --- /dev/null +++ b/app/assets/images/mfa-options/security_key_mobile.svg @@ -0,0 +1 @@ + \ No newline at end of file diff --git a/app/views/users/webauthn_setup/new.html.erb b/app/views/users/webauthn_setup/new.html.erb index 9546190cdfa..90cb3d3c4f7 100644 --- a/app/views/users/webauthn_setup/new.html.erb +++ b/app/views/users/webauthn_setup/new.html.erb @@ -60,9 +60,9 @@ <% end %> <%= c.with_item(heading: t('forms.webauthn_setup.step_2')) do %> <% if mobile? %> - <%= image_tag asset_url('mfa-options/security_key_mobile.gif'), width: 420, height: 193, class: 'height-auto rounded-xl', alt: t('forms.webauthn_setup.step_2_image_mobile_alt') %> + <%= image_tag asset_url('mfa-options/security_key_mobile.svg'), width: 420, height: 193, class: 'height-auto', alt: t('forms.webauthn_setup.step_2_image_mobile_alt'), role: 'img' %> <% else %> - <%= image_tag asset_url('mfa-options/security_key.gif'), width: 420, height: 193, class: 'height-auto rounded-xl', alt: t('forms.webauthn_setup.step_2_image_alt') %> + <%= image_tag asset_url('mfa-options/security_key.svg'), width: 420, height: 193, class: 'height-auto', alt: t('forms.webauthn_setup.step_2_image_alt'), role: 'img' %> <% end %> <% end %> <%= c.with_item(heading: t('forms.webauthn_setup.step_3')) do %> diff --git a/spec/views/users/webauthn_setup/new.html.erb_spec.rb b/spec/views/users/webauthn_setup/new.html.erb_spec.rb index a9777c8f588..d8477ef4f7d 100644 --- a/spec/views/users/webauthn_setup/new.html.erb_spec.rb +++ b/spec/views/users/webauthn_setup/new.html.erb_spec.rb @@ -140,7 +140,7 @@ it 'displays the security key image' do render - expect(rendered).to match(/src=".*security_key-.*\.gif"/) + expect(rendered).to match(/src=".*security_key-.*\.svg"/) end context 'when on a mobile device' do @@ -151,7 +151,7 @@ it 'displays the mobile security key image' do render - expect(rendered).to match(/src=".*security_key_mobile-.*\.gif"/) + expect(rendered).to match(/src=".*security_key_mobile-.*\.svg"/) end end end