Browse Source

[PS-1222] Remove `appBlurClick` throughout the popup and web code (#3208)

* Remove appBlurClick throughout the popup code

`appBlurClick` leads to focus being lost/reset for assistive technology users. It should not be necessary in any case - if focus does need to move after an action, explicitly set it somewhere programmatically using `focus()` rather than relying on browser heuristics

* Remove now redundant blur-click directive
pull/3326/head
Patrick H. Lauke 3 years ago committed by GitHub
parent
commit
babfa30c68
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 9
      apps/browser/src/popup/vault/vault-select.component.html
  2. 2
      apps/web/src/app/components/password-reprompt.component.html
  3. 12
      libs/angular/src/directives/blur-click.directive.ts
  4. 3
      libs/angular/src/jslib.module.ts

9
apps/browser/src/popup/vault/vault-select.component.html

@ -26,26 +26,19 @@ @@ -26,26 +26,19 @@
>
<button
appStopClick
appBlurClick
(click)="selectAllVaults()"
[ngClass]="{ active: !myVaultOnly && !selectOrganizationId }"
>
<i class="bwi bwi-fw bwi-filter" aria-hidden="true"></i>
&nbsp;{{ "allVaults" | i18n }}
</button>
<button
*ngIf="!enforcePersonalOwnwership"
appStopClick
appBlurClick
(click)="selectMyVault()"
>
<button *ngIf="!enforcePersonalOwnwership" appStopClick (click)="selectMyVault()">
<i class="bwi bwi-fw bwi-user" aria-hidden="true"></i>
&nbsp;{{ "myVault" | i18n }}
</button>
<button
*ngFor="let organization of organizations"
appStopClick
appBlurClick
(click)="selectOrganization(organization)"
>
<i

2
apps/web/src/app/components/password-reprompt.component.html

@ -41,7 +41,7 @@ @@ -41,7 +41,7 @@
</div>
</div>
<div class="modal-footer">
<button type="submit" class="btn btn-primary btn-submit" appBlurClick>
<button type="submit" class="btn btn-primary btn-submit">
<span>{{ "ok" | i18n }}</span>
</button>
<button type="button" class="btn btn-outline-secondary" data-dismiss="modal">

12
libs/angular/src/directives/blur-click.directive.ts

@ -1,12 +0,0 @@ @@ -1,12 +0,0 @@
import { Directive, ElementRef, HostListener } from "@angular/core";
@Directive({
selector: "[appBlurClick]",
})
export class BlurClickDirective {
constructor(private el: ElementRef) {}
@HostListener("click") onClick() {
this.el.nativeElement.blur();
}
}

3
libs/angular/src/jslib.module.ts

@ -11,7 +11,6 @@ import { A11yInvalidDirective } from "./directives/a11y-invalid.directive"; @@ -11,7 +11,6 @@ import { A11yInvalidDirective } from "./directives/a11y-invalid.directive";
import { A11yTitleDirective } from "./directives/a11y-title.directive";
import { ApiActionDirective } from "./directives/api-action.directive";
import { AutofocusDirective } from "./directives/autofocus.directive";
import { BlurClickDirective } from "./directives/blur-click.directive";
import { BoxRowDirective } from "./directives/box-row.directive";
import { FallbackSrcDirective } from "./directives/fallback-src.directive";
import { InputStripSpacesDirective } from "./directives/input-strip-spaces.directive";
@ -48,7 +47,6 @@ import { PasswordStrengthComponent } from "./shared/components/password-strength @@ -48,7 +47,6 @@ import { PasswordStrengthComponent } from "./shared/components/password-strength
ApiActionDirective,
AutofocusDirective,
AvatarComponent,
BlurClickDirective,
BoxRowDirective,
CalloutComponent,
ColorPasswordCountPipe,
@ -78,7 +76,6 @@ import { PasswordStrengthComponent } from "./shared/components/password-strength @@ -78,7 +76,6 @@ import { PasswordStrengthComponent } from "./shared/components/password-strength
AutofocusDirective,
AvatarComponent,
BitwardenToastModule,
BlurClickDirective,
BoxRowDirective,
CalloutComponent,
ColorPasswordCountPipe,

Loading…
Cancel
Save