Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 0 additions & 13 deletions app/assets/images/tooltip.svg

This file was deleted.

2 changes: 0 additions & 2 deletions app/assets/stylesheets/_vendor.scss
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
@import 'hint.css/hint';

@import 'basscss-sass/defaults';
@import 'basscss-sass/base-reset';
@import 'basscss-sass/base-forms';
Expand Down
50 changes: 0 additions & 50 deletions app/assets/stylesheets/components/_tooltip.scss

This file was deleted.

1 change: 0 additions & 1 deletion app/assets/stylesheets/components/all.scss
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
@import 'profile-section';
@import 'personal-key';
@import 'position';
@import 'tooltip';
@import 'util';
@import 'verification-badge';
@import 'spinner-button';
Expand Down
10 changes: 0 additions & 10 deletions app/helpers/application_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -15,16 +15,6 @@ def step_class(step, active)
end
end

def tooltip(text)
content_tag(
:span, \
image_tag(asset_url('tooltip.svg'), width: 16, class: 'px1 img-tooltip'), \
class: 'hint--top hint--no-animate', \
'aria-label': text, \
'tabindex': '0',
)
end

def sp_session
session.fetch(:sp, {})
end
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@ def header
def help_text
t("instructions.mfa.#{two_factor_authentication_method}.confirm_code_html",
email: content_tag(:strong, user_email),
app: content_tag(:strong, APP_NAME),
tooltip: view.tooltip(t('tooltips.authentication_app')))
app: content_tag(:strong, APP_NAME))
end

def fallback_question
Expand Down
2 changes: 1 addition & 1 deletion config/locales/instructions/en.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ en:
authenticator:
confirm_code_html: Enter the code from your authenticator app. If you have
several accounts set up in your app, enter the code corresponding to %{email}
at %{app}.%{tooltip}
at %{app}.
manual_entry: Or enter this code manually into your authentication app
piv_cac:
account_not_found:
Expand Down
2 changes: 1 addition & 1 deletion config/locales/instructions/es.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ es:
authenticator:
confirm_code_html: Ingrese el código de su app de autenticación. Si tiene
varias cuentas configuradas en su app, ingrese el código correspondiente
a %{email} en %{app}.%{tooltip}
a %{email} en %{app}.
manual_entry: O ingrese este código manualmente en su aplicación de autenticación
piv_cac:
account_not_found:
Expand Down
2 changes: 1 addition & 1 deletion config/locales/instructions/fr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ fr:
authenticator:
confirm_code_html: Entrez le code à partir de votre application d'authentification.
Si vous avez plusieurs comptes configurés dans votre application, entrez
le code correspondant à %{email} à %{app}.%{tooltip}
le code correspondant à %{email} à %{app}.
manual_entry: Ou entrez ce code manuellement dans votre application d'authentification
piv_cac:
account_not_found:
Expand Down
6 changes: 0 additions & 6 deletions config/locales/tooltips/en.yml

This file was deleted.

6 changes: 0 additions & 6 deletions config/locales/tooltips/es.yml

This file was deleted.

6 changes: 0 additions & 6 deletions config/locales/tooltips/fr.yml

This file was deleted.

1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@
"clipboard": "^1.6.1",
"domready": "^1.0.8",
"focus-trap": "^6.1.3",
"hint.css": "^2.3.2",
"identity-style-guide": "^3.0.0",
"intl-tel-input": "^16.0.7",
"libphonenumber-js": "^1.7.26",
Expand Down
12 changes: 0 additions & 12 deletions spec/helpers/application_helper_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,6 @@
end
end

describe '#tooltip' do
it 'creates a span containing aria label with text and image' do
tooltip_text = 'foobar'

html = helper.tooltip(tooltip_text)

expect(html).to have_css('.hint--top')
expect(html).to have_selector('img')
expect(html).to have_xpath("//span[@aria-label='#{tooltip_text}']")
end
end

describe '#session_with_trust?' do
context 'no user present and page is not one with trust' do
before do
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,11 +52,6 @@
)
end

it 'displays a helpful tooltip to the user' do
tooltip = t('tooltips.authentication_app')
expect(rendered).to have_xpath("//span[@aria-label=\"#{tooltip}\"]")
end

context 'user is reauthenticating' do
before do
allow(@presenter).to receive(:reauthn).and_return(true)
Expand Down
5 changes: 0 additions & 5 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -4680,11 +4680,6 @@ hex-color-regex@^1.1.0:
resolved "https://registry.yarnpkg.com/hex-color-regex/-/hex-color-regex-1.1.0.tgz#4c06fccb4602fe2602b3c93df82d7e7dbf1a8a8e"
integrity sha512-l9sfDFsuqtOqKDsQdqrMRk0U85RZc0RtOR9yPI7mRVOa4FsR/BVnZ0shmQRM96Ji99kYZP/7hn1cedc1+ApsTQ==

hint.css@^2.3.2:
version "2.6.0"
resolved "https://registry.yarnpkg.com/hint.css/-/hint.css-2.6.0.tgz#cd6bbe25e2bc16a6454765d9559ad7bd035b3cf5"
integrity sha512-SnCLQOoPQt4fuHbOOvEL3UZwL0SPFtKMN4YIxAvBTe4oFUtBAT4R2Kk1uk3x5hgN+0xzOb6LFEQf5StbYfNm8w==

hmac-drbg@^1.0.0:
version "1.0.1"
resolved "https://registry.yarnpkg.com/hmac-drbg/-/hmac-drbg-1.0.1.tgz#d2745701025a6c775a6c545793ed502fc0c649a1"
Expand Down