From d528d72b4d363c202d2a50d88d41b8ab0ba4542e Mon Sep 17 00:00:00 2001 From: "Patrick H. Lauke" Date: Sat, 31 Jul 2021 20:03:51 +0100 Subject: [PATCH 1/9] Change links to buttons - more semantically accurate, will expose these as buttons to assistive technologies - note: while having block-level elements like `
` inside a ` @@ -28,8 +28,8 @@ - - + + @@ -39,7 +39,7 @@ - + diff --git a/src/popup/accounts/home.component.html b/src/popup/accounts/home.component.html index 3cdfa2e99f7..5e37c55e8b6 100644 --- a/src/popup/accounts/home.component.html +++ b/src/popup/accounts/home.component.html @@ -1,14 +1,14 @@ -
-
-
-

{{'loginOrCreateNewAccount' | i18n}}

- {{'login' | i18n}} - - {{'enterpriseSingleSignOn' | i18n}} - - {{'createAccount' | i18n}} -
-
- -  {{'settings' | i18n}} - +
+
+
+

{{'loginOrCreateNewAccount' | i18n}}

+ {{'login' | i18n}} + + {{'createAccount' | i18n}} +
+
+ +  {{'settings' | i18n}} + diff --git a/src/popup/accounts/lock.component.html b/src/popup/accounts/lock.component.html index 173487f001f..16ddbb5a71a 100644 --- a/src/popup/accounts/lock.component.html +++ b/src/popup/accounts/lock.component.html @@ -23,11 +23,11 @@ class="monospaced" [(ngModel)]="masterPassword" required appInputVerbatim>
- - +
@@ -38,11 +38,11 @@
- +

- {{'logOut' | i18n}} +

diff --git a/src/popup/accounts/login.component.html b/src/popup/accounts/login.component.html index 0e760197add..6f0434b89fe 100644 --- a/src/popup/accounts/login.component.html +++ b/src/popup/accounts/login.component.html @@ -28,11 +28,11 @@ class="monospaced" [(ngModel)]="masterPassword" required appInputVerbatim>
- - +
diff --git a/src/popup/accounts/register.component.html b/src/popup/accounts/register.component.html index e9fd86bcad0..4bc4ebc8017 100644 --- a/src/popup/accounts/register.component.html +++ b/src/popup/accounts/register.component.html @@ -36,11 +36,11 @@ [appAutofocus]="email !== ''" appInputVerbatim (input)="updatePasswordStrength()">
- - +
@@ -64,11 +64,11 @@ appInputVerbatim>
- - +
diff --git a/src/popup/accounts/set-password.component.html b/src/popup/accounts/set-password.component.html index 53ebcbd33a8..52c75ac9a75 100644 --- a/src/popup/accounts/set-password.component.html +++ b/src/popup/accounts/set-password.component.html @@ -54,11 +54,11 @@ (input)="updatePasswordStrength()" appInputVerbatim>
- - +
@@ -85,11 +85,11 @@ autocomplete="new-password">
- - +
diff --git a/src/popup/accounts/two-factor-options.component.html b/src/popup/accounts/two-factor-options.component.html index f9405d4837c..15ad88013f1 100644 --- a/src/popup/accounts/two-factor-options.component.html +++ b/src/popup/accounts/two-factor-options.component.html @@ -10,14 +10,14 @@
- + +
diff --git a/src/popup/accounts/two-factor.component.html b/src/popup/accounts/two-factor.component.html index 2e8cd97b063..e667aa37718 100644 --- a/src/popup/accounts/two-factor.component.html +++ b/src/popup/accounts/two-factor.component.html @@ -73,7 +73,7 @@

{{'webAuthnNewTab' | i18n}}

- +

- {{'useAnotherTwoStepMethod' | i18n}} +

- +

diff --git a/src/popup/components/pop-out.component.html b/src/popup/components/pop-out.component.html index 1c34a1c762e..2e8d5581c24 100644 --- a/src/popup/components/pop-out.component.html +++ b/src/popup/components/pop-out.component.html @@ -1,5 +1,5 @@ - diff --git a/src/popup/components/send-list.component.html b/src/popup/components/send-list.component.html index 7862cd640e3..cbedd7fbe9d 100644 --- a/src/popup/components/send-list.component.html +++ b/src/popup/components/send-list.component.html @@ -1,4 +1,4 @@ -
@@ -47,4 +47,4 @@
- + diff --git a/src/popup/generator/password-generator-history.component.html b/src/popup/generator/password-generator-history.component.html index f2d93bc27f5..26c96108883 100644 --- a/src/popup/generator/password-generator-history.component.html +++ b/src/popup/generator/password-generator-history.component.html @@ -1,6 +1,6 @@
- @@ -9,7 +9,7 @@ {{'passwordHistory' | i18n}}
-
@@ -26,10 +26,10 @@
- - +
diff --git a/src/popup/generator/password-generator.component.html b/src/popup/generator/password-generator.component.html index bb5f2552318..6986210e941 100644 --- a/src/popup/generator/password-generator.component.html +++ b/src/popup/generator/password-generator.component.html @@ -19,10 +19,10 @@
- {{'regeneratePassword' | i18n}} - {{'copyPassword' | i18n}} + +
diff --git a/src/popup/send/efflux-dates.component.html b/src/popup/send/efflux-dates.component.html index 57dc270244b..b7b35f3a46d 100644 --- a/src/popup/send/efflux-dates.component.html +++ b/src/popup/send/efflux-dates.component.html @@ -46,9 +46,9 @@
- +
diff --git a/src/popup/send/send-add-edit.component.html b/src/popup/send/send-add-edit.component.html index 3db380c5274..63e04cf332e 100644 --- a/src/popup/send/send-add-edit.component.html +++ b/src/popup/send/send-add-edit.component.html @@ -155,11 +155,11 @@ class="monospaced" [(ngModel)]="password" appInputVerbatim [readonly]="disableSend">
- - +
@@ -204,7 +204,7 @@
-
{{'deleteSend' | i18n}}
- +
diff --git a/src/popup/send/send-groupings.component.html b/src/popup/send/send-groupings.component.html index 959ee3c37a1..34bc2f1bfea 100644 --- a/src/popup/send/send-groupings.component.html +++ b/src/popup/send/send-groupings.component.html @@ -8,7 +8,7 @@
-
@@ -22,7 +22,7 @@

{{'noItemsInList' | i18n}}

- @@ -32,22 +32,22 @@ {{'types' | i18n}}
- + +
diff --git a/src/popup/send/send-type.component.html b/src/popup/send/send-type.component.html index 7fe28aacfc6..d489e5f3a53 100644 --- a/src/popup/send/send-type.component.html +++ b/src/popup/send/send-type.component.html @@ -11,7 +11,7 @@
-
@@ -24,7 +24,7 @@

{{'noItemsInList' | i18n}}

- diff --git a/src/popup/settings/excluded-domains.component.html b/src/popup/settings/excluded-domains.component.html index 7a104492829..50bc0a856e5 100644 --- a/src/popup/settings/excluded-domains.component.html +++ b/src/popup/settings/excluded-domains.component.html @@ -16,9 +16,9 @@
- +
- - +
- {{'newUri' | i18n}} - +
{{'excludedDomainsDesc' | i18n}} diff --git a/src/popup/settings/export.component.html b/src/popup/settings/export.component.html index 759fbec53ce..3badbc72cb5 100644 --- a/src/popup/settings/export.component.html +++ b/src/popup/settings/export.component.html @@ -31,11 +31,11 @@ class="monospaced" [(ngModel)]="masterPassword" required appInputVerbatim appAutofocus>
- - +
diff --git a/src/popup/settings/folder-add-edit.component.html b/src/popup/settings/folder-add-edit.component.html index d7e6f9ea56c..9ebdeb7ab3e 100644 --- a/src/popup/settings/folder-add-edit.component.html +++ b/src/popup/settings/folder-add-edit.component.html @@ -24,7 +24,7 @@
-
{{'deleteFolder' | i18n}}
- +
diff --git a/src/popup/settings/folders.component.html b/src/popup/settings/folders.component.html index 6b12b47d591..6becefcf381 100644 --- a/src/popup/settings/folders.component.html +++ b/src/popup/settings/folders.component.html @@ -9,7 +9,7 @@ {{'folders' | i18n}}
-
@@ -17,8 +17,8 @@
- {{f.name}} +
diff --git a/src/popup/settings/settings.component.html b/src/popup/settings/settings.component.html index dca4e801345..6fe0a69667c 100644 --- a/src/popup/settings/settings.component.html +++ b/src/popup/settings/settings.component.html @@ -50,16 +50,16 @@
-
{{'lockNow' | i18n}}
- - +
@@ -72,41 +72,41 @@
-
{{'changeMasterPassword' | i18n}}
- - +
{{'tools' | i18n}}
-
{{'importItems' | i18n}}
- - +
@@ -116,26 +116,26 @@
{{'options' | i18n}}
-
{{'about' | i18n}}
- - +
{{'rateExtensionDesc' | i18n}}
diff --git a/src/popup/vault/add-edit.component.html b/src/popup/vault/add-edit.component.html index 77418d02a45..ea54027ebf2 100644 --- a/src/popup/vault/add-edit.component.html +++ b/src/popup/vault/add-edit.component.html @@ -56,17 +56,17 @@ - - - +
@@ -90,11 +90,11 @@ name="Card.Number" [(ngModel)]="cipher.card.number" appInputVerbatim>
- - +
@@ -121,11 +121,11 @@ name="Card.Code" [(ngModel)]="cipher.card.code" appInputVerbatim>
- - +
@@ -225,9 +225,9 @@
- +
- - - +
- {{'newUri' | i18n}} - +
@@ -301,17 +301,17 @@
-
{{'attachments' | i18n}}
- - +
@@ -333,9 +333,9 @@
- +
@@ -352,11 +352,11 @@ *ngIf="f.type === fieldType.Boolean" appTrueFalseValue trueValue="true" falseValue="false">
- - +
@@ -364,9 +364,9 @@
- + -
- -
-
-
- {{'username' | i18n}} - - -
-
- - - -
-
-
-
- {{'password' | i18n}} -
- {{cipher.login.maskedPassword}}
-
-
-
- - - - - - - -
-
-
-
- {{'verificationCodeTotp' | i18n}} - {{totpCodeFormatted}} -
- - {{totpSec}} - - - - - - - -
- - - -
-
-
- -
-
- {{'cardholderName' | i18n}} - {{cipher.card.cardholderName}} -
-
-
- {{'number' | i18n}} - {{cipher.card.maskedNumber}} - {{cipher.card.number}} -
-
- - - - - - -
-
-
- {{'brand' | i18n}} - {{cipher.card.brand}} -
-
- {{'expiration' | i18n}} - {{cipher.card.expiration}} -
-
-
- {{'securityCode' | i18n}} - {{cipher.card.maskedCode}} - {{cipher.card.code}} -
-
- - - - - - -
-
-
- -
-
- {{'identityName' | i18n}} - {{cipher.identity.fullName}} -
-
- {{'username' | i18n}} - {{cipher.identity.username}} -
-
- {{'company' | i18n}} - {{cipher.identity.company}} -
-
- {{'ssn' | i18n}} - {{cipher.identity.ssn}} -
-
- {{'passportNumber' | i18n}} - {{cipher.identity.passportNumber}} -
-
- {{'licenseNumber' | i18n}} - {{cipher.identity.licenseNumber}} -
-
- {{'email' | i18n}} - {{cipher.identity.email}} -
-
- {{'phone' | i18n}} - {{cipher.identity.phone}} -
-
- {{'address' | i18n}} -
{{cipher.identity.address1}}
-
{{cipher.identity.address2}}
-
{{cipher.identity.address3}}
-
{{cipher.identity.fullAddressPart2}}
-
{{cipher.identity.country}}
-
-
-
-
-
-
-
-
- {{'uri' | i18n}} - {{'website' | i18n}} - - - -
-
- - - - - - -
-
-
-
-
-
- {{'notes' | i18n}} -
-
-
- -
-
-
-
-
- {{'customFields' | i18n}} -
-
-
-
- {{field.name}} -
- {{field.value || ' '}} -
-
- {{field.value}} - {{field.maskedValue}} -
-
- - - {{field.value}} -
-
-
- - - - - - -
-
-
-
-
-
- {{'attachments' | i18n}} -
-
- - {{attachment.fileName}} - {{attachment.sizeName}} - - - -
-
-
-
- -
- - {{'autoFill' | i18n}} -
- - -
- - {{'autoFillAndSave' | i18n}} -
- - -
- - {{'cloneItem' | i18n}} -
- - -
- - {{'moveToOrganization' | i18n}} -
- - -
- - {{'restoreItem' | i18n}} -
- - -
- - {{(cipher.isDeleted ? 'permanentlyDeleteItem' : 'deleteItem') | i18n}} -
- -
-
-
-
-
- {{'dateUpdated' | i18n}}: - {{cipher.revisionDate | date:'medium'}} -
-
- {{'datePasswordUpdated' | i18n}}: - {{cipher.passwordRevisionDisplayDate | date:'medium'}} -
-
- {{'passwordHistory' | i18n}}: - - {{cipher.passwordHistory.length}} - -
-
-
- +
+
+ +
+
+ {{'viewItem' | i18n}} +
+
+ +
+
+ +
+
+ {{'itemInformation' | i18n}} +
+
+
+ {{'name' | i18n}} + +
+ +
+
+
+ {{'username' | i18n}} + + +
+
+ +
+
+
+
+ {{'password' | i18n}} +
+ {{cipher.login.maskedPassword}}
+
+
+
+ + + +
+
+
+
+ {{'verificationCodeTotp' | i18n}} + {{totpCodeFormatted}} +
+ + {{totpSec}} + + + + + + + +
+ +
+
+
+ +
+
+ {{'cardholderName' | i18n}} + {{cipher.card.cardholderName}} +
+
+
+ {{'number' | i18n}} + {{cipher.card.maskedNumber}} + {{cipher.card.number}} +
+
+ + +
+
+
+ {{'brand' | i18n}} + {{cipher.card.brand}} +
+
+ {{'expiration' | i18n}} + {{cipher.card.expiration}} +
+
+
+ {{'securityCode' | i18n}} + {{cipher.card.maskedCode}} + {{cipher.card.code}} +
+
+ + +
+
+
+ +
+
+ {{'identityName' | i18n}} + {{cipher.identity.fullName}} +
+
+ {{'username' | i18n}} + {{cipher.identity.username}} +
+
+ {{'company' | i18n}} + {{cipher.identity.company}} +
+
+ {{'ssn' | i18n}} + {{cipher.identity.ssn}} +
+
+ {{'passportNumber' | i18n}} + {{cipher.identity.passportNumber}} +
+
+ {{'licenseNumber' | i18n}} + {{cipher.identity.licenseNumber}} +
+
+ {{'email' | i18n}} + {{cipher.identity.email}} +
+
+ {{'phone' | i18n}} + {{cipher.identity.phone}} +
+
+ {{'address' | i18n}} +
{{cipher.identity.address1}}
+
{{cipher.identity.address2}}
+
{{cipher.identity.address3}}
+
{{cipher.identity.fullAddressPart2}}
+
{{cipher.identity.country}}
+
+
+
+
+
+
+
+
+ {{'uri' | i18n}} + {{'website' | i18n}} + + + +
+
+ + +
+
+
+
+
+
+ {{'notes' | i18n}} +
+
+
+ +
+
+
+
+
+ {{'customFields' | i18n}} +
+
+
+
+ {{field.name}} +
+ {{field.value || ' '}} +
+
+ {{field.value}} + {{field.maskedValue}} +
+
+ + + {{field.value}} +
+
+
+ + +
+
+
+
+
+
+ {{'attachments' | i18n}} +
+
+ +
+
+
+
+ + + + + + +
+
+
+
+
+ {{'dateUpdated' | i18n}}: + {{cipher.revisionDate | date:'medium'}} +
+
+ {{'datePasswordUpdated' | i18n}}: + {{cipher.passwordRevisionDisplayDate | date:'medium'}} +
+
+ {{'passwordHistory' | i18n}}: + + {{cipher.passwordHistory.length}} + +
+
+
+ From 7e1c4911245f7a4b7584f25b7bf71f5acc1f3617 Mon Sep 17 00:00:00 2001 From: "Patrick H. Lauke" Date: Sat, 31 Jul 2021 21:10:37 +0100 Subject: [PATCH 2/9] Add extra styles for buttons - to maintain the existing look and feel after switching links to buttons --- src/popup/scss/buttons.scss | 186 +++++++++++++++++++----------------- 1 file changed, 98 insertions(+), 88 deletions(-) diff --git a/src/popup/scss/buttons.scss b/src/popup/scss/buttons.scss index 1be16c912e4..f312aa4629e 100644 --- a/src/popup/scss/buttons.scss +++ b/src/popup/scss/buttons.scss @@ -1,88 +1,98 @@ -@import "variables.scss"; - -.btn { - border-radius: $border-radius; - padding: 7px 15px; - border: 1px solid #000000; - font-size: $font-size-base; - white-space: nowrap; - text-align: center; - cursor: pointer; - - @include themify($themes) { - background-color: themed('buttonBackgroundColor'); - border-color: themed('buttonBorderColor'); - color: themed('buttonColor'); - } - - &.primary { - @include themify($themes) { - color: themed('buttonPrimaryColor'); - } - } - - &.danger { - @include themify($themes) { - color: themed('buttonDangerColor'); - } - } - - &:hover:not([disabled]) { - cursor: pointer; - - @include themify($themes) { - background-color: darken(themed('buttonBackgroundColor'), 1.5%); - border-color: darken(themed('buttonBorderColor'), 17%); - color: darken(themed('buttonColor'), 10%); - } - - &.primary { - @include themify($themes) { - color: darken(themed('buttonPrimaryColor'), 6%); - } - } - - &.danger { - @include themify($themes) { - color: darken(themed('buttonDangerColor'), 6%); - } - } - } - - &:focus:not([disabled]) { - cursor: pointer; - outline: 0; - - @include themify($themes) { - background-color: darken(themed('buttonBackgroundColor'), 6%); - border-color: darken(themed('buttonBorderColor'), 25%); - } - } - - &[disabled] { - opacity: 0.65; - cursor: default !important; - } - - &.block { - display: block; - width: 100%; - } - - &.link { - border: none !important; - background: none !important; - - &:focus { - text-decoration: underline; - } - } -} - -.action-buttons { - .btn { - &:focus { - outline: auto; - } - } -} +@import "variables.scss"; + +.btn { + border-radius: $border-radius; + padding: 7px 15px; + border: 1px solid #000000; + font-size: $font-size-base; + white-space: nowrap; + text-align: center; + cursor: pointer; + + @include themify($themes) { + background-color: themed('buttonBackgroundColor'); + border-color: themed('buttonBorderColor'); + color: themed('buttonColor'); + } + + &.primary { + @include themify($themes) { + color: themed('buttonPrimaryColor'); + } + } + + &.danger { + @include themify($themes) { + color: themed('buttonDangerColor'); + } + } + + &:hover:not([disabled]) { + cursor: pointer; + + @include themify($themes) { + background-color: darken(themed('buttonBackgroundColor'), 1.5%); + border-color: darken(themed('buttonBorderColor'), 17%); + color: darken(themed('buttonColor'), 10%); + } + + &.primary { + @include themify($themes) { + color: darken(themed('buttonPrimaryColor'), 6%); + } + } + + &.danger { + @include themify($themes) { + color: darken(themed('buttonDangerColor'), 6%); + } + } + } + + &:focus:not([disabled]) { + cursor: pointer; + outline: 0; + + @include themify($themes) { + background-color: darken(themed('buttonBackgroundColor'), 6%); + border-color: darken(themed('buttonBorderColor'), 25%); + } + } + + &[disabled] { + opacity: 0.65; + cursor: default !important; + } + + &.block { + display: block; + width: 100%; + } + + &.link { + border: none !important; + background: none !important; + + &:focus { + text-decoration: underline; + } + } +} + +.action-buttons { + .btn { + &:focus { + outline: auto; + } + } +} + +button.box-content-row { + display: block; + width: 100%; + background: none; +} + +button { + border: none; +} \ No newline at end of file From fdb3e25d9827cb25d6cc0e3c989371235277a5a5 Mon Sep 17 00:00:00 2001 From: "Patrick H. Lauke" Date: Thu, 2 Sep 2021 22:06:36 +0100 Subject: [PATCH 3/9] Apply suggestions from code review Co-authored-by: Addison Beck --- src/popup/accounts/home.component.html | 3 ++- src/popup/accounts/lock.component.html | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/popup/accounts/home.component.html b/src/popup/accounts/home.component.html index 5e37c55e8b6..53c9384f12f 100644 --- a/src/popup/accounts/home.component.html +++ b/src/popup/accounts/home.component.html @@ -3,7 +3,8 @@

{{'loginOrCreateNewAccount' | i18n}}

{{'login' | i18n}} - {{'createAccount' | i18n}} diff --git a/src/popup/accounts/lock.component.html b/src/popup/accounts/lock.component.html index 16ddbb5a71a..1b7da04eaea 100644 --- a/src/popup/accounts/lock.component.html +++ b/src/popup/accounts/lock.component.html @@ -42,7 +42,7 @@

- +

From 8ce5cbe94373109d75bdcbe58bc571460f8a3b5e Mon Sep 17 00:00:00 2001 From: "Patrick H. Lauke" Date: Sat, 18 Sep 2021 17:32:02 +0100 Subject: [PATCH 4/9] Tweak styles to accommodate for buttons --- src/notification/bar.html | 2 +- src/notification/bar.scss | 14 ++++++++------ src/popup/scss/box.scss | 10 ++++++++-- src/popup/scss/buttons.scss | 7 +++++-- 4 files changed, 22 insertions(+), 11 deletions(-) diff --git a/src/notification/bar.html b/src/notification/bar.html index fcc34c585df..f50db6e5fd3 100644 --- a/src/notification/bar.html +++ b/src/notification/bar.html @@ -15,7 +15,7 @@ - diff --git a/src/notification/bar.scss b/src/notification/bar.scss index 1f673d52550..bc49e8a2c61 100644 --- a/src/notification/bar.scss +++ b/src/notification/bar.scss @@ -55,31 +55,33 @@ img { margin-right: 10px; } -button:not(.link) { +button:not(.link), +button:not(.neutral) { background-color: #175DDC; padding: 5px 15px; border-radius: 3px; color: #ffffff; border: 0; -} - button:not(.link):hover { + &:hover { cursor: pointer; background-color: #1751bd; } +} -button.link { +button.link, +button.neutral { background: none; padding: 5px 15px; color: #175DDC; border: 0; -} - button.link:hover { + &:hover { cursor: pointer; background: none; text-decoration: underline; } +} body[class*='lang-en'] .add-buttons { width: 175px; diff --git a/src/popup/scss/box.scss b/src/popup/scss/box.scss index c11651c49b0..51930d76f04 100644 --- a/src/popup/scss/box.scss +++ b/src/popup/scss/box.scss @@ -300,7 +300,8 @@ margin-top: 5px; } - > a { + > a, + > button { padding: 8px 8px 8px 4px; margin: 0; @@ -315,6 +316,12 @@ padding-left: 10px; } + &.box-content-row-newmulti { + @include themify($themes) { + color: themed('primaryColor'); + } + } + &.box-content-row-checkbox, &.box-content-row-input, &.box-content-row-slider { label, .row-label { font-size: $font-size-base; @@ -511,7 +518,6 @@ justify-content: center; align-items: center; min-width: 34px; - height: 100%; margin-left: -5px; @include themify($themes) { diff --git a/src/popup/scss/buttons.scss b/src/popup/scss/buttons.scss index f312aa4629e..f6f3718b8fd 100644 --- a/src/popup/scss/buttons.scss +++ b/src/popup/scss/buttons.scss @@ -69,7 +69,8 @@ width: 100%; } - &.link { + &.link, + &.neutral { border: none !important; background: none !important; @@ -90,9 +91,11 @@ button.box-content-row { display: block; width: 100%; - background: none; + text-align: left; } button { border: none; + background: transparent; + color: inherit; } \ No newline at end of file From c178cbdc129d41b14685b8728a13af22526b1b23 Mon Sep 17 00:00:00 2001 From: "Patrick H. Lauke" Date: Sat, 18 Sep 2021 17:45:54 +0100 Subject: [PATCH 5/9] Resolve merge conflict --- src/notification/bar.html | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/notification/bar.html b/src/notification/bar.html index f50db6e5fd3..d352b2133c9 100644 --- a/src/notification/bar.html +++ b/src/notification/bar.html @@ -27,9 +27,12 @@ + + + - - + + From f389ea29515f83ee3a5720b957cbf423c94c755c Mon Sep 17 00:00:00 2001 From: "Patrick H. Lauke" Date: Sat, 18 Sep 2021 17:55:18 +0100 Subject: [PATCH 6/9] Change links to buttons in new components/fragments that appeared after latest merge --- src/popup/accounts/update-temp-password.component.html | 8 ++++---- src/popup/components/cipher-row.component.html | 4 ++-- src/popup/components/password-reprompt.component.html | 4 ++-- src/popup/components/set-pin.component.html | 4 ++-- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/popup/accounts/update-temp-password.component.html b/src/popup/accounts/update-temp-password.component.html index 91ce4d5cbf5..608e7214c8f 100644 --- a/src/popup/accounts/update-temp-password.component.html +++ b/src/popup/accounts/update-temp-password.component.html @@ -36,11 +36,11 @@ appInputVerbatim (input)="updatePasswordStrength()">
- - +
@@ -62,11 +62,11 @@ appInputVerbatim>
- - +
diff --git a/src/popup/components/cipher-row.component.html b/src/popup/components/cipher-row.component.html index 818e44c1b8b..91aa472d422 100644 --- a/src/popup/components/cipher-row.component.html +++ b/src/popup/components/cipher-row.component.html @@ -1,4 +1,4 @@ -
@@ -20,4 +20,4 @@ - + diff --git a/src/popup/components/password-reprompt.component.html b/src/popup/components/password-reprompt.component.html index 0033cdd2864..24354cb16ed 100644 --- a/src/popup/components/password-reprompt.component.html +++ b/src/popup/components/password-reprompt.component.html @@ -12,11 +12,11 @@ class="monospaced" [(ngModel)]="masterPassword" required appAutofocus>
- - +
diff --git a/src/popup/components/set-pin.component.html b/src/popup/components/set-pin.component.html index 8513011282e..c9a0126a8a8 100644 --- a/src/popup/components/set-pin.component.html +++ b/src/popup/components/set-pin.component.html @@ -14,11 +14,11 @@ class="monospaced" [(ngModel)]="pin" required appInputVerbatim cdkFocusInitial>
- - +
From bf7dd5da966a9ee2ea613056ed733150471db03b Mon Sep 17 00:00:00 2001 From: "Patrick H. Lauke" Date: Tue, 28 Sep 2021 19:55:11 +0100 Subject: [PATCH 7/9] Change links to buttons in new split-out components --- .../vault/add-edit-custom-fields.component.html | 12 ++++++------ src/popup/vault/view-custom-fields.component.html | 8 ++++---- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/popup/vault/add-edit-custom-fields.component.html b/src/popup/vault/add-edit-custom-fields.component.html index 843fde75d0f..4fe2eae4228 100644 --- a/src/popup/vault/add-edit-custom-fields.component.html +++ b/src/popup/vault/add-edit-custom-fields.component.html @@ -8,9 +8,9 @@
- +
@@ -28,11 +28,11 @@
- - +
@@ -41,9 +41,9 @@
- + -
- -
-
-
- {{'username' | i18n}} - - -
-
- -
-
-
-
- {{'password' | i18n}} -
- {{cipher.login.maskedPassword}}
-
-
-
- - - -
-
-
-
- {{'verificationCodeTotp' | i18n}} - {{totpCodeFormatted}} -
- - {{totpSec}} - - - - - - - -
- -
-
-
- -
-
- {{'cardholderName' | i18n}} - {{cipher.card.cardholderName}} -
-
-
- {{'number' | i18n}} - {{cipher.card.maskedNumber}} - {{cipher.card.number}} -
-
- - -
-
-
- {{'brand' | i18n}} - {{cipher.card.brand}} -
-
- {{'expiration' | i18n}} - {{cipher.card.expiration}} -
-
-
- {{'securityCode' | i18n}} - {{cipher.card.maskedCode}} - {{cipher.card.code}} -
-
- - -
-
-
- -
-
- {{'identityName' | i18n}} - {{cipher.identity.fullName}} -
-
- {{'username' | i18n}} - {{cipher.identity.username}} -
-
- {{'company' | i18n}} - {{cipher.identity.company}} -
-
- {{'ssn' | i18n}} - {{cipher.identity.ssn}} -
-
- {{'passportNumber' | i18n}} - {{cipher.identity.passportNumber}} -
-
- {{'licenseNumber' | i18n}} - {{cipher.identity.licenseNumber}} -
-
- {{'email' | i18n}} - {{cipher.identity.email}} -
-
- {{'phone' | i18n}} - {{cipher.identity.phone}} -
-
- {{'address' | i18n}} -
{{cipher.identity.address1}}
-
{{cipher.identity.address2}}
-
{{cipher.identity.address3}}
-
{{cipher.identity.fullAddressPart2}}
-
{{cipher.identity.country}}
-
-
-
-
-
-
-
-
- {{'uri' | i18n}} - {{'website' | i18n}} - - - -
-
- - -
-
-
-
-
-
- {{'notes' | i18n}} -
-
-
- -
-
-
-
- -
-
-
- {{'attachments' | i18n}} -
-
- -
-
-
-
- < - - - - - -
-
-
-
-
- {{'dateUpdated' | i18n}}: - {{cipher.revisionDate | date:'medium'}} -
-
- {{'datePasswordUpdated' | i18n}}: - {{cipher.passwordRevisionDisplayDate | date:'medium'}} -
-
- {{'passwordHistory' | i18n}}: - - {{cipher.passwordHistory.length}} - -
-
-
- +
+
+ +
+
+ {{'viewItem' | i18n}} +
+
+ +
+
+ +
+
+ {{'itemInformation' | i18n}} +
+
+
+ {{'name' | i18n}} + +
+ +
+
+
+ {{'username' | i18n}} + + +
+
+ +
+
+
+
+ {{'password' | i18n}} +
+ {{cipher.login.maskedPassword}}
+
+
+
+ + + +
+
+
+
+ {{'verificationCodeTotp' | i18n}} + {{totpCodeFormatted}} +
+ + {{totpSec}} + + + + + + + +
+ +
+
+
+ +
+
+ {{'cardholderName' | i18n}} + {{cipher.card.cardholderName}} +
+
+
+ {{'number' | i18n}} + {{cipher.card.maskedNumber}} + {{cipher.card.number}} +
+
+ + +
+
+
+ {{'brand' | i18n}} + {{cipher.card.brand}} +
+
+ {{'expiration' | i18n}} + {{cipher.card.expiration}} +
+
+
+ {{'securityCode' | i18n}} + {{cipher.card.maskedCode}} + {{cipher.card.code}} +
+
+ + +
+
+
+ +
+
+ {{'identityName' | i18n}} + {{cipher.identity.fullName}} +
+
+ {{'username' | i18n}} + {{cipher.identity.username}} +
+
+ {{'company' | i18n}} + {{cipher.identity.company}} +
+
+ {{'ssn' | i18n}} + {{cipher.identity.ssn}} +
+
+ {{'passportNumber' | i18n}} + {{cipher.identity.passportNumber}} +
+
+ {{'licenseNumber' | i18n}} + {{cipher.identity.licenseNumber}} +
+
+ {{'email' | i18n}} + {{cipher.identity.email}} +
+
+ {{'phone' | i18n}} + {{cipher.identity.phone}} +
+
+ {{'address' | i18n}} +
{{cipher.identity.address1}}
+
{{cipher.identity.address2}}
+
{{cipher.identity.address3}}
+
{{cipher.identity.fullAddressPart2}}
+
{{cipher.identity.country}}
+
+
+
+
+
+
+
+
+ {{'uri' | i18n}} + {{'website' | i18n}} + + + +
+
+ + +
+
+
+
+
+
+ {{'notes' | i18n}} +
+
+
+ +
+
+
+
+ +
+
+
+ {{'attachments' | i18n}} +
+
+ +
+
+
+
+ < + + + + + +
+
+
+
+
+ {{'dateUpdated' | i18n}}: + {{cipher.revisionDate | date:'medium'}} +
+
+ {{'datePasswordUpdated' | i18n}}: + {{cipher.passwordRevisionDisplayDate | date:'medium'}} +
+
+ {{'passwordHistory' | i18n}}: + + {{cipher.passwordHistory.length}} + +
+
+
+