fix: kill remaining native confirm() in admin, standardize on themed dialog

Order cancel/refund and user suspend used window.confirm(). Migrated all three to app-confirm-dialog, matching products/categories/orders delete gates from earlier in this phase. Remaining window.confirm() usages are the three canDeactivate dirty guards (categories/products/project-editor) - left as-is, since CanDeactivate needs a synchronous or Observable/Promise return and browser navigation guards conventionally use the native dialog there; converting those is a separate, larger refactor. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
fix: review 'Archive' button was mislabeled delete, zero confirmation
2026-08-13 08:34:18 +04:00 · 2026-08-13 08:28:56 +04:00 · 2026-08-13 08:25:09 +04:00 · 2026-08-13 08:22:01 +04:00 · 2026-08-13 08:19:22 +04:00
14 changed files with 225 additions and 24 deletions
--- a/src/app/features/admin/categories/pages/admin-categories-list-page.component.ts
+++ b/src/app/features/admin/categories/pages/admin-categories-list-page.component.ts
@@ -1,14 +1,18 @@
-import { ChangeDetectionStrategy, Component, inject } from '@angular/core';
+import { ChangeDetectionStrategy, Component, inject, signal } from '@angular/core';
 import { Router } from '@angular/router';
 import { AdminCategoriesFacade } from '../facade/admin-categories.facade';
 import { AdminCategoriesListComponent } from '../components/admin-categories-list.component';
 import { LanguageService } from '../../../../services/language.service';
 import { TranslateService } from '../../../../i18n/translate.service';
+import { ConfirmDialogComponent } from '../../../../shared/ui/confirm-dialog/confirm-dialog.component';
+import { DialogComponent } from '../../../../shared/ui/dialog/dialog.component';
+import { ButtonComponent } from '../../../../shared/ui/button/button.component';
+import { TranslatePipe } from '../../../../i18n/translate.pipe';

@Component({
  selector: 'app-admin-categories-list-page',
  standalone: true,
-  imports: [AdminCategoriesListComponent],
+  imports: [AdminCategoriesListComponent, ConfirmDialogComponent, DialogComponent, ButtonComponent, TranslatePipe],
  template: `<app-admin-categories-list
    [treeRows]="facade.visibleTreeRows()"
    [flatCategories]="facade.filteredCategories()"
@@ -37,14 +41,36 @@ import { TranslateService } from '../../../../i18n/translate.service';
    (selectionChange)="facade.toggleSelection($event.id, $event.checked)"
    (selectAll)="facade.toggleAll($event)"
    (bulkVisibility)="facade.applyBulkVisibility($event)"
-    (bulkDelete)="facade.applyBulkDelete()"
+    (bulkDelete)="bulkDelete()"
    (bulkDuplicate)="facade.applyBulkDuplicate()"
    (bulkAssignParent)="facade.applyBulkAssignParent($event)"
    (bulkAssignImage)="facade.applyBulkAssignImage($event)"
    (bulkExport)="facade.exportSelectedAsCsv()"
    (viewModeChange)="facade.setViewMode($event)"
    (densityChange)="facade.setDensity($event)"
-    (columnToggle)="facade.setColumnVisible($event.column, $event.visible)" />`,
+    (columnToggle)="facade.setColumnVisible($event.column, $event.visible)" />
+    <app-confirm-dialog
+      [open]="!!pendingDeleteId()"
+      [titleText]="'adminCategories.deleteTitle' | translate"
+      [message]="'adminCategories.confirmDelete' | translate"
+      [destructive]="true"
+      (confirmed)="confirmDelete()"
+      (cancelled)="pendingDeleteId.set(null)" />
+    <app-confirm-dialog
+      [open]="bulkDeleteConfirmOpen()"
+      [titleText]="'adminCategories.deleteTitle' | translate"
+      [message]="'adminCategories.confirmBulkDelete' | translate"
+      [destructive]="true"
+      (confirmed)="confirmBulkDelete()"
+      (cancelled)="bulkDeleteConfirmOpen.set(false)" />
+    @if (deleteBlockedMessage()) {
+      <app-dialog [open]="true" [titleText]="'adminCategories.deleteBlocked' | translate" size="sm" (closed)="deleteBlockedMessage.set(null)">
+        <p>{{ deleteBlockedMessage() }}</p>
+        <div class="app-confirm-dialog__actions">
+          <app-button variant="primary" (click)="deleteBlockedMessage.set(null)">{{ 'common.confirm' | translate }}</app-button>
+        </div>
+      </app-dialog>
+    }`,
  changeDetection: ChangeDetectionStrategy.OnPush
 })
 export class AdminCategoriesListPageComponent {
@@ -57,6 +83,10 @@ export class AdminCategoriesListPageComponent {
  readonly isExpandedFn = (id: string) => this.facade.isExpanded(id);
  readonly canDeleteFn = (id: string) => this.facade.canDelete(id);

+  readonly pendingDeleteId = signal<string | null>(null);
+  readonly bulkDeleteConfirmOpen = signal(false);
+  readonly deleteBlockedMessage = signal<string | null>(null);
+
  constructor() {
    this.facade.loadList();
    this.facade.loadDashboardStats();
@@ -75,12 +105,27 @@ export class AdminCategoriesListPageComponent {

  deleteOne(id: string): void {
    if (!this.facade.canDelete(id)) {
-      window.alert(this.translate.t('adminCategories.deleteBlocked'));
+      this.deleteBlockedMessage.set(this.translate.t('adminCategories.deleteBlocked'));
      return;
    }
-    if (window.confirm(this.translate.t('adminCategories.confirmDelete'))) {
+    this.pendingDeleteId.set(id);
+  }
+
+  confirmDelete(): void {
+    const id = this.pendingDeleteId();
+    if (id) {
      this.facade.deleteOne(id);
    }
+    this.pendingDeleteId.set(null);
+  }
+
+  bulkDelete(): void {
+    this.bulkDeleteConfirmOpen.set(true);
+  }
+
+  confirmBulkDelete(): void {
+    this.facade.applyBulkDelete();
+    this.bulkDeleteConfirmOpen.set(false);
  }

  private lang(): string {
--- a/src/app/features/admin/moderation/pages/admin-reviews-list-page.component.html
+++ b/src/app/features/admin/moderation/pages/admin-reviews-list-page.component.html
@@ -53,9 +53,16 @@
      <app-button variant="secondary" size="sm" (click)="applyBulkStatus()">{{ 'adminOrders.applyStatus' | translate }}</app-button>
      <app-button variant="secondary" size="sm" (click)="facade.applyBulkVisible(false)">{{ 'adminModeration.hideSelected' | translate }}</app-button>
      <app-button variant="secondary" size="sm" (click)="facade.exportSelectedAsCsv()">{{ 'adminProducts.bulkExportAction' | translate }}</app-button>
-      <app-button variant="danger" size="sm" (click)="facade.applyBulkDelete()">{{ 'adminModeration.archiveSelected' | translate }}</app-button>
+      <app-button variant="danger" size="sm" (click)="bulkDelete()">{{ 'adminModeration.deleteSelected' | translate }}</app-button>
    </div>
  }
+  <app-confirm-dialog
+    [open]="bulkDeleteConfirmOpen()"
+    [titleText]="'adminModeration.deleteSelected' | translate"
+    [message]="'adminModeration.confirmBulkDelete' | translate"
+    [destructive]="true"
+    (confirmed)="confirmBulkDelete()"
+    (cancelled)="bulkDeleteConfirmOpen.set(false)" />

  @if (facade.loading()) {
    <div class="skeleton-rows" role="status" aria-live="polite" aria-busy="true">
--- a/src/app/features/admin/moderation/pages/admin-reviews-list-page.component.ts
+++ b/src/app/features/admin/moderation/pages/admin-reviews-list-page.component.ts
@@ -16,11 +16,12 @@ import { SkeletonComponent } from '../../../../shared/ui/skeleton/skeleton.compo
 import { CardComponent } from '../../../../shared/ui/card/card.component';
 import { ModerationDashboardComponent } from '../components/moderation-dashboard/moderation-dashboard.component';
 import { ReviewHealthWidgetComponent, ReviewHealthItem } from '../components/review-health-widget/review-health-widget.component';
+import { ConfirmDialogComponent } from '../../../../shared/ui/confirm-dialog/confirm-dialog.component';

@Component({
  selector: 'app-admin-reviews-list-page',
  standalone: true,
-  imports: [CommonModule, FormsModule, TranslatePipe, ButtonComponent, InputComponent, BadgeComponent, TableComponent, PaginationComponent, EmptyStateComponent, SkeletonComponent, CardComponent, ModerationDashboardComponent, ReviewHealthWidgetComponent],
+  imports: [CommonModule, FormsModule, TranslatePipe, ButtonComponent, InputComponent, BadgeComponent, TableComponent, PaginationComponent, EmptyStateComponent, SkeletonComponent, CardComponent, ModerationDashboardComponent, ReviewHealthWidgetComponent, ConfirmDialogComponent],
  templateUrl: './admin-reviews-list-page.component.html',
  styleUrls: ['./admin-reviews-list-page.component.scss'],
  changeDetection: ChangeDetectionStrategy.OnPush
@@ -35,6 +36,7 @@ export class AdminReviewsListPageComponent {
  readonly allColumns = ALL_REVIEW_COLUMNS;
  protected readonly columnsPanelOpen = signal(false);
  protected readonly bulkStatusValue = signal<AdminReviewStatus>('approved');
+  protected readonly bulkDeleteConfirmOpen = signal(false);

  constructor() {
    this.facade.loadList();
@@ -45,6 +47,15 @@ export class AdminReviewsListPageComponent {
    return Math.max(1, Math.ceil(this.facade.total() / this.facade.filters().pageSize));
  }

+  bulkDelete(): void {
+    this.bulkDeleteConfirmOpen.set(true);
+  }
+
+  confirmBulkDelete(): void {
+    this.facade.applyBulkDelete();
+    this.bulkDeleteConfirmOpen.set(false);
+  }
+
  view(id: string): void {
    void this.router.navigate([this.languageService.currentLanguage(), 'backoffice', 'moderation', id]);
  }
--- a/src/app/features/admin/orders/pages/admin-order-detail-page.component.html
+++ b/src/app/features/admin/orders/pages/admin-order-detail-page.component.html
@@ -88,6 +88,20 @@
      <app-button variant="secondary" size="sm" (click)="submitInternalNote(order.id)">{{ 'adminOrders.addNote' | translate }}</app-button>
    </div>
  </section>
+  <app-confirm-dialog
+    [open]="!!pendingCancelId()"
+    [titleText]="'adminOrders.cancelOrder' | translate"
+    [message]="'adminOrders.confirmCancel' | translate"
+    [destructive]="true"
+    (confirmed)="confirmCancel()"
+    (cancelled)="pendingCancelId.set(null)" />
+  <app-confirm-dialog
+    [open]="!!pendingRefundId()"
+    [titleText]="'adminOrders.requestRefund' | translate"
+    [message]="'adminOrders.confirmRefund' | translate"
+    [destructive]="true"
+    (confirmed)="confirmRefund()"
+    (cancelled)="pendingRefundId.set(null)" />
 </main>
 } @else {
  <p>{{ 'common.loading' | translate }}</p>
--- a/src/app/features/admin/orders/pages/admin-order-detail-page.component.ts
+++ b/src/app/features/admin/orders/pages/admin-order-detail-page.component.ts
@@ -10,6 +10,7 @@ import { LanguageService } from '../../../../services/language.service';
 import { ButtonComponent } from '../../../../shared/ui/button/button.component';
 import { BadgeComponent } from '../../../../shared/ui/badge/badge.component';
 import { OrderTimelineComponent, OrderTimelineEntry } from '../components/order-timeline/order-timeline.component';
+import { ConfirmDialogComponent } from '../../../../shared/ui/confirm-dialog/confirm-dialog.component';

 const WORKFLOW_STEPS: AdminOrderStatus[] = ['pending', 'processing', 'shipped', 'delivered'];
 const TERMINAL_STATUSES: AdminOrderStatus[] = ['cancelled', 'refunded'];
@@ -17,7 +18,7 @@ const TERMINAL_STATUSES: AdminOrderStatus[] = ['cancelled', 'refunded'];
@Component({
  selector: 'app-admin-order-detail-page',
  standalone: true,
-  imports: [CommonModule, FormsModule, TranslatePipe, ButtonComponent, BadgeComponent, OrderTimelineComponent],
+  imports: [CommonModule, FormsModule, TranslatePipe, ButtonComponent, BadgeComponent, OrderTimelineComponent, ConfirmDialogComponent],
  templateUrl: './admin-order-detail-page.component.html',
  styleUrls: ['./admin-order-detail-page.component.scss'],
  changeDetection: ChangeDetectionStrategy.OnPush
@@ -34,6 +35,8 @@ export class AdminOrderDetailPageComponent {
  readonly selectableStatuses: AdminOrderStatus[] = this.statuses.filter(status => !TERMINAL_STATUSES.includes(status));
  readonly workflowSteps = WORKFLOW_STEPS;
  readonly noteDraft = signal('');
+  readonly pendingCancelId = signal<string | null>(null);
+  readonly pendingRefundId = signal<string | null>(null);
  readonly internalNoteDraft = signal('');

  readonly isTerminal = computed(() => {
@@ -85,15 +88,27 @@ export class AdminOrderDetailPageComponent {
  }

  cancel(id: string): void {
-    if (window.confirm(this.translate.t('adminOrders.confirmCancel'))) {
+    this.pendingCancelId.set(id);
+  }
+
+  confirmCancel(): void {
+    const id = this.pendingCancelId();
+    if (id) {
      this.facade.cancelOrder(id);
    }
+    this.pendingCancelId.set(null);
  }

  requestRefund(id: string): void {
-    if (window.confirm(this.translate.t('adminOrders.confirmRefund'))) {
+    this.pendingRefundId.set(id);
+  }
+
+  confirmRefund(): void {
+    const id = this.pendingRefundId();
+    if (id) {
      this.facade.requestRefund(id);
    }
+    this.pendingRefundId.set(null);
  }

  submitNote(id: string): void {
--- a/src/app/features/admin/orders/pages/admin-orders-list-page.component.html
+++ b/src/app/features/admin/orders/pages/admin-orders-list-page.component.html
@@ -45,9 +45,16 @@
      <app-button variant="secondary" size="sm" (click)="facade.applyBulkArchive(true)">{{ 'adminOrders.archiveSelected' | translate }}</app-button>
      <app-button variant="secondary" size="sm" (click)="facade.exportSelectedAsCsv()">{{ 'adminProducts.bulkExportAction' | translate }}</app-button>
      <app-button variant="secondary" size="sm" (click)="printSelection()">{{ 'adminOrders.printSelected' | translate }}</app-button>
-      <app-button variant="danger" size="sm" (click)="facade.applyBulkDelete()">{{ 'adminProducts.bulkDelete' | translate }}</app-button>
+      <app-button variant="danger" size="sm" (click)="bulkDelete()">{{ 'adminProducts.bulkDelete' | translate }}</app-button>
    </div>
  }
+  <app-confirm-dialog
+    [open]="bulkDeleteConfirmOpen()"
+    [titleText]="'adminOrders.bulkDeleteTitle' | translate"
+    [message]="'adminOrders.confirmBulkDelete' | translate"
+    [destructive]="true"
+    (confirmed)="confirmBulkDelete()"
+    (cancelled)="bulkDeleteConfirmOpen.set(false)" />

  @if (facade.loading()) {
    <div class="skeleton-rows" role="status" aria-live="polite" aria-busy="true">
--- a/src/app/features/admin/orders/pages/admin-orders-list-page.component.ts
+++ b/src/app/features/admin/orders/pages/admin-orders-list-page.component.ts
@@ -15,11 +15,12 @@ import { EmptyStateComponent } from '../../../../shared/ui/empty-state/empty-sta
 import { SkeletonComponent } from '../../../../shared/ui/skeleton/skeleton.component';
 import { CardComponent } from '../../../../shared/ui/card/card.component';
 import { OrdersDashboardComponent } from '../components/orders-dashboard/orders-dashboard.component';
+import { ConfirmDialogComponent } from '../../../../shared/ui/confirm-dialog/confirm-dialog.component';

@Component({
  selector: 'app-admin-orders-list-page',
  standalone: true,
-  imports: [CommonModule, FormsModule, TranslatePipe, ButtonComponent, InputComponent, BadgeComponent, TableComponent, PaginationComponent, EmptyStateComponent, SkeletonComponent, CardComponent, OrdersDashboardComponent],
+  imports: [CommonModule, FormsModule, TranslatePipe, ButtonComponent, InputComponent, BadgeComponent, TableComponent, PaginationComponent, EmptyStateComponent, SkeletonComponent, CardComponent, OrdersDashboardComponent, ConfirmDialogComponent],
  templateUrl: './admin-orders-list-page.component.html',
  styleUrls: ['./admin-orders-list-page.component.scss'],
  changeDetection: ChangeDetectionStrategy.OnPush
@@ -35,6 +36,7 @@ export class AdminOrdersListPageComponent {
  readonly allColumns = ALL_ORDER_COLUMNS;
  protected readonly columnsPanelOpen = signal(false);
  protected readonly bulkStatusValue = signal<AdminOrderStatus>('pending');
+  protected readonly bulkDeleteConfirmOpen = signal(false);

  constructor() {
    this.facade.loadList();
@@ -61,6 +63,15 @@ export class AdminOrdersListPageComponent {
    this.facade.applyBulkStatus(this.bulkStatusValue());
  }

+  bulkDelete(): void {
+    this.bulkDeleteConfirmOpen.set(true);
+  }
+
+  confirmBulkDelete(): void {
+    this.facade.applyBulkDelete();
+    this.bulkDeleteConfirmOpen.set(false);
+  }
+
  printSelection(): void {
    window.print();
  }
--- a/src/app/features/admin/products/pages/admin-products-list-page.component.ts
+++ b/src/app/features/admin/products/pages/admin-products-list-page.component.ts
@@ -1,13 +1,15 @@
-import { ChangeDetectionStrategy, Component, inject } from '@angular/core';
+import { ChangeDetectionStrategy, Component, inject, signal } from '@angular/core';
 import { Router } from '@angular/router';
 import { AdminProductsFacade } from '../facade/admin-products.facade';
 import { AdminProductsListComponent } from '../components/admin-products-list.component';
 import { LanguageService } from '../../../../services/language.service';
+import { ConfirmDialogComponent } from '../../../../shared/ui/confirm-dialog/confirm-dialog.component';
+import { TranslatePipe } from '../../../../i18n/translate.pipe';

@Component({
  selector: 'app-admin-products-list-page',
  standalone: true,
-  imports: [AdminProductsListComponent],
+  imports: [AdminProductsListComponent, ConfirmDialogComponent, TranslatePipe],
  template: `<app-admin-products-list
    [products]="facade.products()"
    [categories]="facade.categories()"
@@ -25,7 +27,7 @@ import { LanguageService } from '../../../../services/language.service';
    (create)="create()"
    (edit)="edit($event)"
    (duplicate)="duplicate($event)"
-    (delete)="facade.deleteOne($event)"
+    (delete)="deleteOne($event)"
    (archive)="facade.archiveOne($event)"
    (restore)="facade.restoreOne($event)"
    (loadMore)="facade.loadMore()"
@@ -33,14 +35,28 @@ import { LanguageService } from '../../../../services/language.service';
    (selectionChange)="facade.toggleSelection($event.id, $event.checked)"
    (selectAll)="facade.toggleAll($event)"
    (bulkVisibility)="facade.applyBulkVisibility($event)"
-    (bulkDelete)="facade.applyBulkDelete()"
+    (bulkDelete)="bulkDelete()"
    (bulkDuplicate)="facade.applyBulkDuplicate()"
    (bulkAssignCategory)="facade.applyBulkAssignCategory($event)"
    (bulkAssignTags)="facade.applyBulkAssignTags($event)"
    (bulkExport)="facade.exportSelectedAsCsv()"
    (viewModeChange)="facade.setViewMode($event)"
    (densityChange)="facade.setDensity($event)"
-    (columnToggle)="facade.setColumnVisible($event.column, $event.visible)" />`,
+    (columnToggle)="facade.setColumnVisible($event.column, $event.visible)" />
+    <app-confirm-dialog
+      [open]="!!pendingDeleteId()"
+      [titleText]="'adminProducts.deleteTitle' | translate"
+      [message]="'adminProducts.confirmDelete' | translate"
+      [destructive]="true"
+      (confirmed)="confirmDelete()"
+      (cancelled)="pendingDeleteId.set(null)" />
+    <app-confirm-dialog
+      [open]="bulkDeleteConfirmOpen()"
+      [titleText]="'adminProducts.deleteTitle' | translate"
+      [message]="'adminProducts.confirmBulkDelete' | translate"
+      [destructive]="true"
+      (confirmed)="confirmBulkDelete()"
+      (cancelled)="bulkDeleteConfirmOpen.set(false)" />`,
  changeDetection: ChangeDetectionStrategy.OnPush
 })
 export class AdminProductsListPageComponent {
@@ -50,6 +66,9 @@ export class AdminProductsListPageComponent {

  readonly healthFn = (product: Parameters<AdminProductsFacade['health']>[0]) => this.facade.health(product);

+  readonly pendingDeleteId = signal<string | null>(null);
+  readonly bulkDeleteConfirmOpen = signal(false);
+
  constructor() {
    this.facade.loadCategories();
    this.facade.loadList();
@@ -60,6 +79,27 @@ export class AdminProductsListPageComponent {
  edit(id: string): void { this.facade.loadForEdit(id, 'edit'); void this.router.navigate([this.lang(), 'backoffice', 'products', id, 'edit']); }
  duplicate(id: string): void { this.facade.loadForEdit(id, 'duplicate'); void this.router.navigate([this.lang(), 'backoffice', 'products', id, 'duplicate']); }

+  deleteOne(id: string): void {
+    this.pendingDeleteId.set(id);
+  }
+
+  confirmDelete(): void {
+    const id = this.pendingDeleteId();
+    if (id) {
+      this.facade.deleteOne(id);
+    }
+    this.pendingDeleteId.set(null);
+  }
+
+  bulkDelete(): void {
+    this.bulkDeleteConfirmOpen.set(true);
+  }
+
+  confirmBulkDelete(): void {
+    this.facade.applyBulkDelete();
+    this.bulkDeleteConfirmOpen.set(false);
+  }
+
  private lang(): string {
    return this.languageService.currentLanguage();
  }
--- a/src/app/features/admin/users/pages/admin-users-page.component.html
+++ b/src/app/features/admin/users/pages/admin-users-page.component.html
@@ -120,4 +120,12 @@
      <p>{{ entry.timestamp | date:'short' }} — {{ ('adminUsers.actor.' + entry.actor) | translate }} — {{ auditText(entry) }}</p>
    }
  </app-dialog>
+
+  <app-confirm-dialog
+    [open]="!!pendingSuspendUserId()"
+    [titleText]="'adminUsers.suspend' | translate"
+    [message]="'adminUsers.confirmSuspend' | translate"
+    [destructive]="true"
+    (confirmed)="confirmSuspend()"
+    (cancelled)="pendingSuspendUserId.set(null)" />
 </section>
--- a/src/app/features/admin/users/pages/admin-users-page.component.ts
+++ b/src/app/features/admin/users/pages/admin-users-page.component.ts
@@ -12,11 +12,12 @@ import { TableComponent } from '../../../../shared/ui/table/table.component';
 import { DialogComponent } from '../../../../shared/ui/dialog/dialog.component';
 import { SkeletonComponent } from '../../../../shared/ui/skeleton/skeleton.component';
 import { EmptyStateComponent } from '../../../../shared/ui/empty-state/empty-state.component';
+import { ConfirmDialogComponent } from '../../../../shared/ui/confirm-dialog/confirm-dialog.component';

@Component({
  selector: 'app-admin-users-page',
  standalone: true,
-  imports: [CommonModule, FormsModule, TranslatePipe, ButtonComponent, InputComponent, BadgeComponent, TableComponent, DialogComponent, SkeletonComponent, EmptyStateComponent],
+  imports: [CommonModule, FormsModule, TranslatePipe, ButtonComponent, InputComponent, BadgeComponent, TableComponent, DialogComponent, SkeletonComponent, EmptyStateComponent, ConfirmDialogComponent],
  templateUrl: './admin-users-page.component.html',
  styleUrls: ['./admin-users-page.component.scss'],
  changeDetection: ChangeDetectionStrategy.OnPush
@@ -28,6 +29,7 @@ export class AdminUsersPageComponent {
  readonly inviteEmail = signal('');
  readonly inviteRoleId = signal('viewer');
  readonly inviteScope = signal<AdminUserScope>('office');
+  readonly pendingSuspendUserId = signal<string | null>(null);

  constructor() {
    this.facade.loadAll();
@@ -40,12 +42,21 @@ export class AdminUsersPageComponent {

  toggleStatus(userId: string, current: AdminUserStatus): void {
    const next: AdminUserStatus = current === 'suspended' ? 'active' : 'suspended';
-    if (next === 'suspended' && !window.confirm(this.translate.t('adminUsers.confirmSuspend'))) {
+    if (next === 'suspended') {
+      this.pendingSuspendUserId.set(userId);
      return;
    }
    this.facade.setStatus(userId, next);
  }

+  confirmSuspend(): void {
+    const userId = this.pendingSuspendUserId();
+    if (userId) {
+      this.facade.setStatus(userId, 'suspended');
+    }
+    this.pendingSuspendUserId.set(null);
+  }
+
  private static readonly PERMISSION_KEYS: Record<string, string> = {
    '*': 'all',
    'products.manage': 'productsManage',
--- a/src/app/i18n/en.ts
+++ b/src/app/i18n/en.ts
@@ -1302,7 +1302,8 @@ export const en: Translations = {
    notAvailable: 'Not available yet',
    changeStatus: 'Change status',
    hideSelected: 'Hide',
-    archiveSelected: 'Archive',
+    deleteSelected: 'Delete',
+    confirmBulkDelete: 'Delete the selected reviews? This cannot be undone.',
    reportsQueue: 'Reports queue',
    reportsColumn: 'Reports',
    reportTarget: 'Reported item',
@@ -1360,6 +1361,8 @@ export const en: Translations = {
    },
  },
  adminOrders: {
+    bulkDeleteTitle: 'Delete orders',
+    confirmBulkDelete: 'Delete the selected orders? This cannot be undone.',
    back: 'Back to orders',
    search: 'Search by order number, name, email, or phone…',
    export: 'Export',
@@ -1549,6 +1552,8 @@ export const en: Translations = {
    saveDraft: 'Save draft',
    publish: 'Publish',
    confirmDelete: 'Delete this category? This cannot be undone.',
+    confirmBulkDelete: 'Delete the selected categories? This cannot be undone.',
+    deleteTitle: 'Delete category',
    confirmLeaveUnsaved: 'You have unsaved changes. Leave without saving?',
    deleteBlocked: 'This category can\'t be deleted while it has subcategories or assigned products.',
    emptyTitle: 'No categories yet',
@@ -1615,6 +1620,9 @@ export const en: Translations = {
  },
  adminProducts: {
    confirmLeaveUnsaved: 'You have unsaved changes. Leave without saving?',
+    deleteTitle: 'Delete product',
+    confirmDelete: 'Delete this product? This cannot be undone.',
+    confirmBulkDelete: 'Delete the selected products? This cannot be undone.',
    emptyTitle: 'No products found',
    emptyDescription: 'Try adjusting your filters, or create a new product.',
    emptyGuide: 'Good products have a clear title, at least one photo, a price, and a short description — that\'s enough to publish. You can always add more detail later.',
--- a/src/app/i18n/hy.ts
+++ b/src/app/i18n/hy.ts
@@ -1297,7 +1297,8 @@ export const hy: Translations = {
    notAvailable: 'Դեռ հասանելի չէ',
    changeStatus: 'Փոխել կարգավիճակը',
    hideSelected: 'Թաքցնել',
-    archiveSelected: 'Արխիվացնել',
+    deleteSelected: 'Ջնջել',
+    confirmBulkDelete: 'Ջնջե՞լ ընտրված կարծիքները։ Հնարավոր չէ հետարկել։',
    reportsQueue: 'Բողոքների հերթ',
    reportsColumn: 'Բողոքներ',
    reportTarget: 'Բողոքարկված օբյեկտ',
@@ -1355,6 +1356,8 @@ export const hy: Translations = {
    },
  },
  adminOrders: {
+    bulkDeleteTitle: 'Ջնջել պատվերները',
+    confirmBulkDelete: 'Ջնջե՞լ ընտրված պատվերները։ Հնարավոր չէ հետարկել։',
    back: 'Վերադառնալ պատվերներին',
    search: 'Փնտրել ըստ պատվերի համարի, անվան, էլ. փոստի կամ հեռախոսի…',
    export: 'Արտահանել',
@@ -1544,6 +1547,8 @@ export const hy: Translations = {
    saveDraft: 'Պահպանել սևագիրը',
    publish: 'Հրապարակել',
    confirmDelete: 'Ջնջե՞լ այս կատեգորիան։ Հնարավոր չէ հետարկել։',
+    confirmBulkDelete: 'Ջնջե՞լ ընտրված կատեգորիաները։ Հնարավոր չէ հետարկել։',
+    deleteTitle: 'Ջնջել կատեգորիան',
    confirmLeaveUnsaved: 'Դուք ունեք չպահպանված փոփոխություններ։ Դո՞ւրս գալ առանց պահպանելու։',
    deleteBlocked: 'Այս կատեգորիան հնարավոր չէ ջնջել, քանի դեռ ունի ենթակատեգորիաներ կամ նշանակված ապրանքներ։',
    emptyTitle: 'Կատեգորիաներ դեռ չկան',
@@ -1610,6 +1615,9 @@ export const hy: Translations = {
  },
  adminProducts: {
    confirmLeaveUnsaved: 'Դուք ունեք չպահպանված փոփոխություններ։ Դո՞ւրս գալ առանց պահպանելու։',
+    deleteTitle: 'Ջնջել ապրանքը',
+    confirmDelete: 'Ջնջե՞լ այս ապրանքը։ Հնարավոր չէ հետարկել։',
+    confirmBulkDelete: 'Ջնջե՞լ ընտրված ապրանքները։ Հնարավոր չէ հետարկել։',
    emptyTitle: 'Ապրանքներ չեն գտնվել',
    emptyDescription: 'Փոխեք ֆիլտրերը կամ ստեղծեք նոր ապրանք։',
    emptyGuide: 'Լավ ապրանքին բավական է հստակ վերնագիր, առնվազն մեկ լուսանկար, գին և կարճ նկարագրություն՝ հրապարակելու համար։ Մնացածը կարող եք ավելացնել հետո։',
--- a/src/app/i18n/ru.ts
+++ b/src/app/i18n/ru.ts
@@ -1297,7 +1297,8 @@ export const ru: Translations = {
    notAvailable: 'Пока недоступно',
    changeStatus: 'Изменить статус',
    hideSelected: 'Скрыть',
-    archiveSelected: 'В архив',
+    deleteSelected: 'Удалить',
+    confirmBulkDelete: 'Удалить выбранные отзывы? Это действие нельзя отменить.',
    reportsQueue: 'Очередь жалоб',
    reportsColumn: 'Жалобы',
    reportTarget: 'Объект жалобы',
@@ -1355,6 +1356,8 @@ export const ru: Translations = {
    },
  },
  adminOrders: {
+    bulkDeleteTitle: 'Удалить заказы',
+    confirmBulkDelete: 'Удалить выбранные заказы? Это действие нельзя отменить.',
    back: 'Назад к заказам',
    search: 'Поиск по номеру заказа, имени, email или телефону…',
    export: 'Экспорт',
@@ -1544,6 +1547,8 @@ export const ru: Translations = {
    saveDraft: 'Сохранить черновик',
    publish: 'Опубликовать',
    confirmDelete: 'Удалить эту категорию? Это действие нельзя отменить.',
+    confirmBulkDelete: 'Удалить выбранные категории? Это действие нельзя отменить.',
+    deleteTitle: 'Удалить категорию',
    confirmLeaveUnsaved: 'У вас есть несохранённые изменения. Выйти без сохранения?',
    deleteBlocked: 'Эту категорию нельзя удалить, пока у неё есть подкатегории или назначенные товары.',
    emptyTitle: 'Категорий пока нет',
@@ -1610,6 +1615,9 @@ export const ru: Translations = {
  },
  adminProducts: {
    confirmLeaveUnsaved: 'У вас есть несохранённые изменения. Выйти без сохранения?',
+    deleteTitle: 'Удалить товар',
+    confirmDelete: 'Удалить этот товар? Это действие нельзя отменить.',
+    confirmBulkDelete: 'Удалить выбранные товары? Это действие нельзя отменить.',
    emptyTitle: 'Товары не найдены',
    emptyDescription: 'Измените фильтры или создайте новый товар.',
    emptyGuide: 'Хорошему товару нужны понятное название, минимум одно фото, цена и короткое описание — этого достаточно для публикации. Остальное можно добавить позже.',
--- a/src/app/i18n/translations.ts
+++ b/src/app/i18n/translations.ts
@@ -1301,7 +1301,8 @@ export interface Translations {
    notAvailable: string;
    changeStatus: string;
    hideSelected: string;
-    archiveSelected: string;
+    deleteSelected: string;
+    confirmBulkDelete: string;
    reportsQueue: string;
    reportsColumn: string;
    reportTarget: string;
@@ -1359,6 +1360,8 @@ export interface Translations {
    };
  };
  adminOrders: {
+    bulkDeleteTitle: string;
+    confirmBulkDelete: string;
    back: string;
    search: string;
    export: string;
@@ -1557,6 +1560,8 @@ export interface Translations {
    saveDraft: string;
    publish: string;
    confirmDelete: string;
+    confirmBulkDelete: string;
+    deleteTitle: string;
    confirmLeaveUnsaved: string;
    deleteBlocked: string;
    emptyTitle: string;
@@ -1623,6 +1628,9 @@ export interface Translations {
  };
  adminProducts: {
    confirmLeaveUnsaved: string;
+    deleteTitle: string;
+    confirmDelete: string;
+    confirmBulkDelete: string;
    emptyTitle: string;
    emptyDescription: string;
    emptyGuide: string;
Author	SHA1	Message	Date
sdarbinyan	c461d9bd5f	fix: kill remaining native confirm() in admin, standardize on themed dialog Some checks failed Architecture Governance / architecture (push) Has been cancelled Details Order cancel/refund and user suspend used window.confirm(). Migrated all three to app-confirm-dialog, matching products/categories/orders delete gates from earlier in this phase. Remaining window.confirm() usages are the three canDeactivate dirty guards (categories/products/project-editor) - left as-is, since CanDeactivate needs a synchronous or Observable/Promise return and browser navigation guards conventionally use the native dialog there; converting those is a separate, larger refactor. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 08:34:18 +04:00
sdarbinyan	63039b8707	fix: review 'Archive' button was mislabeled delete, zero confirmation The bulk button read 'Archive' but called applyBulkDelete() -> gateway.deleteReview() - a real hard delete, and reviews have no archived status in the model at all, so 'archive' was never a real concept here. Relabeled to 'Delete' and added an app-confirm-dialog gate, same pattern as products/categories/orders. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 08:28:56 +04:00
sdarbinyan	908f10e022	fix: order bulk-delete had zero confirmation Bulk-delete button called facade.applyBulkDelete() with no gate. Added app-confirm-dialog, same pattern as products/categories. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 08:25:09 +04:00
sdarbinyan	49aab63124	fix: product delete (single+bulk) had zero confirmation (delete) and (bulkDelete) called facade.deleteOne()/applyBulkDelete() straight from the click event. Added app-confirm-dialog gates for both, same pattern as categories. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 08:22:01 +04:00
sdarbinyan	320f1f44b7	fix: category bulk-delete had no confirmation, single-delete used native dialogs Bulk-delete called facade.applyBulkDelete() straight from the button with zero confirmation. Single-delete had a confirm gate but via raw window.confirm/alert instead of the themed dialog used elsewhere. Migrated both to app-confirm-dialog (single + new bulk), and the delete-blocked message to a themed app-dialog instead of window.alert. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>	2026-08-13 08:19:22 +04:00