Skip to content

Commit fb0ca2e

Browse files
CXIEP-7764: cr changes pt2
1 parent 828ff5e commit fb0ca2e

File tree

5 files changed

+25
-31
lines changed

5 files changed

+25
-31
lines changed

integration-libs/s4om/components/order-detail-attachments/attachments-dialog/s4om-attachments-dialog.component.html

+21-24
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,13 @@
3636
<!-- Modal Body -->
3737
<div class="cx-dialog-body modal-body">
3838
@for (errorId of erroredAttachments; track errorId) {
39-
<div class="cx-dialog-message attachment-error" aria-live="assertive" aria-atomic="true">
40-
<cx-message
41-
[text]="'s4omOrder.attachments.dialog.downloadError' | cxTranslate"
42-
[type]="globalMessageType.MSG_TYPE_ERROR"
43-
(closeMessage)="closeErrorMessage(errorId)"
44-
>
45-
</cx-message>
46-
</div>
39+
<cx-message
40+
[text]="'s4omOrder.attachments.dialog.downloadError' | cxTranslate"
41+
[type]="globalMessageType.MSG_TYPE_ERROR"
42+
(closeMessage)="closeErrorMessage(errorId)"
43+
class="cx-dialog-message attachment-error"
44+
>
45+
</cx-message>
4746
}
4847
@if (attachments$ | async; as attachments) {
4948
@if (!(error$ | async)) {
@@ -85,25 +84,23 @@
8584
</tr>
8685
}
8786
} @empty {
88-
<div class="cx-dialog-message info-message" aria-live="assertive" aria-atomic="true">
89-
<cx-message
90-
[text]="'s4omOrder.attachments.dialog.empty' | cxTranslate"
91-
[type]="globalMessageType.MSG_TYPE_INFO"
92-
[isVisibleCloseButton]="false"
93-
>
94-
</cx-message>
95-
</div>
87+
<cx-message
88+
[text]="'s4omOrder.attachments.dialog.empty' | cxTranslate"
89+
[type]="globalMessageType.MSG_TYPE_INFO"
90+
[isVisibleCloseButton]="false"
91+
class="cx-dialog-message info-message"
92+
>
93+
</cx-message>
9694
}
9795
</table>
9896
} @else {
99-
<div class="cx-dialog-message error-message" aria-live="assertive" aria-atomic="true">
100-
<cx-message
101-
[text]="'s4omOrder.attachments.dialog.error' | cxTranslate"
102-
[type]="globalMessageType.MSG_TYPE_ERROR"
103-
[isVisibleCloseButton]="false"
104-
>
105-
</cx-message>
106-
</div>
97+
<cx-message
98+
[text]="'s4omOrder.attachments.dialog.error' | cxTranslate"
99+
[type]="globalMessageType.MSG_TYPE_ERROR"
100+
[isVisibleCloseButton]="false"
101+
class="cx-dialog-message error-message"
102+
>
103+
</cx-message>
107104
}
108105
} @else {
109106
<div class="cx-spinner">

integration-libs/s4om/components/order-detail-attachments/attachments-dialog/s4om-attachments-dialog.component.ts

+3-2
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ export class S4omAttachmentsDialogComponent {
103103
link.href = blobUrl;
104104
link.target = '_blank';
105105
link.rel = 'noopener noreferrer';
106-
link.download = fileName as string;
106+
link.download = fileName;
107107
document.body.appendChild(link);
108108
link.click();
109109
document.body.removeChild(link);
@@ -127,7 +127,8 @@ export class S4omAttachmentsDialogComponent {
127127
}
128128

129129
addErrorMessage(attachmentId: string): void {
130-
this.erroredAttachments.push(`${attachmentId}_${this.errorCounter++}`);
130+
this.erroredAttachments.push(`${attachmentId}_${this.errorCounter}`);
131+
this.errorCounter++;
131132
this.cd.detectChanges();
132133
}
133134

integration-libs/s4om/root/model/augmented-core.model.ts

-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
import '@spartacus/cart/base/root';
88
import { ScheduleLine } from './schedule-line.model';
9-
109
import { LAUNCH_CALLER } from '@spartacus/storefront';
1110

1211
declare module '@spartacus/storefront' {

integration-libs/s4om/styles/_attachments-dialog.scss

-4
Original file line numberDiff line numberDiff line change
@@ -71,10 +71,6 @@ cx-s4om-order-attachments-dialog {
7171
box-shadow: 0 0 0 0.2rem rgba(0, 123, 255, 0.25);
7272
}
7373
}
74-
75-
//.cx-dialog-message {
76-
// padding-right: 30px;
77-
//}
7874
}
7975

8076
.cx-dialog-footer {

projects/schematics/src/dependencies.json

+1
Original file line numberDiff line numberDiff line change
@@ -457,6 +457,7 @@
457457
"@angular/core": "^19.1.8",
458458
"@spartacus/cart": "2211.38.0-1",
459459
"@spartacus/core": "2211.38.0-1",
460+
"@spartacus/order": "2211.38.0-1",
460461
"@spartacus/pdf-invoices": "2211.38.0-1",
461462
"@spartacus/requested-delivery-date": "2211.38.0-1",
462463
"@spartacus/schematics": "2211.38.0-1",

0 commit comments

Comments
 (0)