1
0
mirror of https://github.com/bitwarden/browser synced 2026-02-05 19:23:19 +00:00

remove emoji and use correct change detection

This commit is contained in:
Bryan Cunningham
2026-01-09 11:54:28 -05:00
parent 7e22fd5fca
commit 4d99f1ec59
2 changed files with 5 additions and 7 deletions

View File

@@ -26,6 +26,7 @@ import { ariaDisableElement } from "../utils";
export class MenuItemComponent implements ButtonLikeAbstraction {
readonly disabled = model<boolean>(false);
readonly loading = model<boolean>(false);
readonly elementRef = inject(ElementRef<HTMLButtonElement>);
protected readonly disabledAttr = computed(() => {

View File

@@ -1,6 +1,6 @@
import { OverlayModule } from "@angular/cdk/overlay";
import { NgTemplateOutlet } from "@angular/common";
import { Component } from "@angular/core";
import { ChangeDetectionStrategy, Component } from "@angular/core";
import { Meta, StoryObj, moduleMetadata } from "@storybook/angular";
import { I18nService } from "@bitwarden/common/platform/abstractions/i18n.service";
@@ -19,15 +19,14 @@ const template = /*html*/ `
</div>
<bit-menu #myMenu>
<button type="button" bitMenuItem [bitAction]="action">Perform action {{ statusEmoji }}</button>
<button type="button" bitMenuItem [bitAction]="action">Perform action</button>
</bit-menu>
`;
// FIXME(https://bitwarden.atlassian.net/browse/CL-764): Migrate to OnPush
// eslint-disable-next-line @angular-eslint/prefer-on-push-component-change-detection
@Component({
template,
selector: "app-promise-example",
changeDetection: ChangeDetectionStrategy.OnPush,
imports: [
NgTemplateOutlet,
AsyncActionsModule,
@@ -38,12 +37,10 @@ const template = /*html*/ `
],
})
class PromiseExampleComponent {
statusEmoji = "🟡";
action = async () => {
await new Promise<void>((resolve, reject) => {
await new Promise<void>((resolve) => {
setTimeout(() => {
resolve();
this.statusEmoji = "🟢";
}, 5000);
});
};