diff --git a/npm/ng-packs/apps/dev-app/src/app/app-routing.module.ts b/npm/ng-packs/apps/dev-app/src/app/app-routing.module.ts index 6588538b28..0d9f82ea25 100644 --- a/npm/ng-packs/apps/dev-app/src/app/app-routing.module.ts +++ b/npm/ng-packs/apps/dev-app/src/app/app-routing.module.ts @@ -31,4 +31,4 @@ const routes: Routes = [ imports: [RouterModule.forRoot(routes, {})], exports: [RouterModule], }) -export class AppRoutingModule { } +export class AppRoutingModule {} diff --git a/npm/ng-packs/apps/dev-app/src/app/app.module.ts b/npm/ng-packs/apps/dev-app/src/app/app.module.ts index b2dc1ba780..f144639db4 100644 --- a/npm/ng-packs/apps/dev-app/src/app/app.module.ts +++ b/npm/ng-packs/apps/dev-app/src/app/app.module.ts @@ -39,7 +39,7 @@ import { APP_ROUTE_PROVIDER } from './route.provider'; ThemeLeptonXModule.forRoot(), SideMenuLayoutModule.forRoot(), AccountLayoutModule.forRoot(), - InternetConnectionStatusComponent + InternetConnectionStatusComponent, ], providers: [APP_ROUTE_PROVIDER], declarations: [AppComponent], diff --git a/npm/ng-packs/packages/core/src/lib/services/internet-connection-service.ts b/npm/ng-packs/packages/core/src/lib/services/internet-connection-service.ts index d74e65d036..5a8ad4bd63 100644 --- a/npm/ng-packs/packages/core/src/lib/services/internet-connection-service.ts +++ b/npm/ng-packs/packages/core/src/lib/services/internet-connection-service.ts @@ -5,28 +5,28 @@ import { BehaviorSubject } from 'rxjs'; @Injectable({ providedIn: 'root', }) -export class InternetConnectionService{ - readonly document = inject(DOCUMENT) +export class InternetConnectionService { + readonly document = inject(DOCUMENT); readonly window = this.document.defaultView; readonly navigator = this.window.navigator; - private status$ = new BehaviorSubject(this.navigator.onLine) + private status$ = new BehaviorSubject(this.navigator.onLine); private status = signal(this.navigator.onLine); - networkStatus = computed(() => this.status()) - - constructor(){ + networkStatus = computed(() => this.status()); + + constructor() { this.window.addEventListener('offline', () => this.setStatus(false)); this.window.addEventListener('online', () => this.setStatus(true)); } - setStatus(val:boolean){ - this.status.set(val) - this.status$.next(val) + setStatus(val: boolean) { + this.status.set(val); + this.status$.next(val); } - get networkStatus$(){ - return this.status$.asObservable() + get networkStatus$() { + return this.status$.asObservable(); } } diff --git a/npm/ng-packs/packages/theme-shared/src/lib/components/http-error-wrapper/http-error-wrapper.component.html b/npm/ng-packs/packages/theme-shared/src/lib/components/http-error-wrapper/http-error-wrapper.component.html index cb3adb0e17..f2a887da85 100644 --- a/npm/ng-packs/packages/theme-shared/src/lib/components/http-error-wrapper/http-error-wrapper.component.html +++ b/npm/ng-packs/packages/theme-shared/src/lib/components/http-error-wrapper/http-error-wrapper.component.html @@ -4,32 +4,28 @@ class="error" [style.backgroundColor]="backgroundColor" > - + @if (!hideCloseIcon) { + + } -
-
-
-

{{ statusText }} {{ title | abpLocalization }}

-
- {{ details | abpLocalization }} -
-
- - {{ { key: '::Menu:Home', defaultValue: 'Home' } | abpLocalization }} - + @if (!customComponent) { +
+
+
+

{{ statusText }} {{ title | abpLocalization }}

+
+ {{ details | abpLocalization }} +
+
-
+ }
diff --git a/npm/ng-packs/packages/theme-shared/src/lib/components/http-error-wrapper/http-error-wrapper.component.ts b/npm/ng-packs/packages/theme-shared/src/lib/components/http-error-wrapper/http-error-wrapper.component.ts index 7761751e8c..83002a3a88 100644 --- a/npm/ng-packs/packages/theme-shared/src/lib/components/http-error-wrapper/http-error-wrapper.component.ts +++ b/npm/ng-packs/packages/theme-shared/src/lib/components/http-error-wrapper/http-error-wrapper.component.ts @@ -1,38 +1,40 @@ import { ApplicationRef, Component, - Injector, inject, OnInit, - ComponentFactoryResolver, ElementRef, EmbeddedViewRef, Type, ViewChild, AfterViewInit, OnDestroy, + createComponent, + EnvironmentInjector, + DestroyRef, } from '@angular/core'; +import { takeUntilDestroyed } from '@angular/core/rxjs-interop'; import { DOCUMENT } from '@angular/common'; +import { Router } from '@angular/router'; import { fromEvent, Subject } from 'rxjs'; import { debounceTime, filter } from 'rxjs/operators'; -import { LocalizationParam, SubscriptionService } from '@abp/ng.core'; +import { LocalizationParam } from '@abp/ng.core'; import { ErrorScreenErrorCodes } from '../../models'; @Component({ selector: 'abp-http-error-wrapper', templateUrl: './http-error-wrapper.component.html', styleUrls: ['http-error-wrapper.component.scss'], - providers: [SubscriptionService], }) export class HttpErrorWrapperComponent implements OnInit, AfterViewInit, OnDestroy { + protected readonly destroyRef = inject(DestroyRef); protected readonly document = inject(DOCUMENT); protected readonly window = this.document.defaultView; + protected readonly router = inject(Router); appRef!: ApplicationRef; - cfRes!: ComponentFactoryResolver; - - injector!: Injector; + environmentInjector!: EnvironmentInjector; status: ErrorScreenErrorCodes = 0; @@ -57,21 +59,23 @@ export class HttpErrorWrapperComponent implements OnInit, AfterViewInit, OnDestr return this.status ? `[${this.status}]` : ''; } - constructor(private subscription: SubscriptionService) {} - ngOnInit(): void { - this.backgroundColor = - this.window.getComputedStyle(this.document.body)?.getPropertyValue('background-color') || '#fff'; + const computedStyle = this.window.getComputedStyle(this.document.body); + const backgroundColor = computedStyle?.getPropertyValue('background-color'); + this.backgroundColor = backgroundColor || '#fff'; } ngAfterViewInit(): void { if (this.customComponent) { - const customComponentRef = this.cfRes - .resolveComponentFactory(this.customComponent) - .create(this.injector); + const customComponentRef = createComponent(this.customComponent, { + environmentInjector: this.environmentInjector, + }); + customComponentRef.instance.errorStatus = this.status; customComponentRef.instance.destroy$ = this.destroy$; + this.appRef.attachView(customComponentRef.hostView); + if (this.containerRef) { this.containerRef.nativeElement.appendChild( (customComponentRef.hostView as EmbeddedViewRef).rootNodes[0], @@ -80,14 +84,17 @@ export class HttpErrorWrapperComponent implements OnInit, AfterViewInit, OnDestr customComponentRef.changeDetectorRef.detectChanges(); } - const keyup$ = fromEvent(this.document, 'keyup').pipe( - debounceTime(150), - filter((key: KeyboardEvent) => key && key.key === 'Escape'), - ); - this.subscription.addOne(keyup$, () => this.destroy()); + fromEvent(this.document, 'keyup') + .pipe( + debounceTime(150), + filter((key: KeyboardEvent) => key && key.key === 'Escape'), + takeUntilDestroyed(this.destroyRef), + ) + .subscribe(() => this.destroy()); } - ngOnDestroy(): void { + goHome(): void { + this.router.navigateByUrl('/', { onSameUrlNavigation: 'reload' }); this.destroy(); } @@ -95,4 +102,8 @@ export class HttpErrorWrapperComponent implements OnInit, AfterViewInit, OnDestr this.destroy$.next(); this.destroy$.complete(); } + + ngOnDestroy(): void { + this.destroy(); + } } diff --git a/npm/ng-packs/packages/theme-shared/src/lib/components/internet-connection-status/internet-connection-status.component.ts b/npm/ng-packs/packages/theme-shared/src/lib/components/internet-connection-status/internet-connection-status.component.ts index e649555f40..9c59947891 100644 --- a/npm/ng-packs/packages/theme-shared/src/lib/components/internet-connection-status/internet-connection-status.component.ts +++ b/npm/ng-packs/packages/theme-shared/src/lib/components/internet-connection-status/internet-connection-status.component.ts @@ -1,47 +1,60 @@ -import { Component, computed, inject } from '@angular/core'; -import { NgIf } from '@angular/common' -import { InternetConnectionService , LocalizationModule } from '@abp/ng.core'; +import { Component, inject } from '@angular/core'; +import { InternetConnectionService, LocalizationModule } from '@abp/ng.core'; +import { NgbTooltip } from '@ng-bootstrap/ng-bootstrap'; @Component({ selector: 'abp-internet-status', standalone: true, - imports:[NgIf, LocalizationModule], + imports: [LocalizationModule, NgbTooltip], template: ` -
- - -
- `, - styles: [` - .blink { - animation: blinker 0.9s cubic-bezier(.5, 0, 1, 1) infinite alternate; - } - @keyframes blinker { - 0% { color:#c1c1c1 } - 70% { color: #DC3545 } - 100% { color: #DC3545 } - } - - .text-blinking{ - font-size:1.2rem; + @if (!isOnline()) { +
+ + +
} + `, + styles: [ + ` + .blink { + animation: blinker 0.9s cubic-bezier(0.5, 0, 1, 1) infinite alternate; + } + @keyframes blinker { + 0% { + color: #c1c1c1; + } + 70% { + color: #fa2379; + } + 100% { + color: #fa2379; + } + } - .status-icon{ - position: fixed; - z-index: 999999; - top: 10px; - right: 10px; - } + .text-blinking { + font-size: 30px; + } - @media (width < 768px){ - .status-icon{ - top: 26px; - right: 134px; + .status-icon { + position: fixed; + z-index: 999999; + top: 50%; + left: 50%; + width: 30px; + text-align: center; + margin-left: -15px; + margin-top: -15px; + translate: transform(-50%, -50%); } - } - `] + `, + ], }) -export class InternetConnectionStatusComponent{ +export class InternetConnectionStatusComponent { internetConnectionService = inject(InternetConnectionService); - isOnline = this.internetConnectionService.networkStatus + isOnline = this.internetConnectionService.networkStatus; } diff --git a/npm/ng-packs/packages/theme-shared/src/lib/constants/default-errors.ts b/npm/ng-packs/packages/theme-shared/src/lib/constants/default-errors.ts index 46a0605b95..6a2797e037 100644 --- a/npm/ng-packs/packages/theme-shared/src/lib/constants/default-errors.ts +++ b/npm/ng-packs/packages/theme-shared/src/lib/constants/default-errors.ts @@ -51,3 +51,17 @@ export const CUSTOM_HTTP_ERROR_HANDLER_PRIORITY = Object.freeze({ high: 9, veryHigh: 99, }); + +export const HTTP_ERROR_STATUS = { + '401': 'AbpUi::401Message', + '403': 'AbpUi::403Message', + '404': 'AbpUi::404Message', + '500': 'AbpUi::500Message', +}; + +export const HTTP_ERROR_DETAIL = { + '401': 'AbpUi::DefaultErrorMessage401Detail', + '403': 'AbpUi::DefaultErrorMessage403Detail', + '404': 'AbpUi::DefaultErrorMessage404Detail', + '500': 'AbpUi::DefaultErrorMessage', +}; diff --git a/npm/ng-packs/packages/theme-shared/src/lib/constants/index.ts b/npm/ng-packs/packages/theme-shared/src/lib/constants/index.ts new file mode 100644 index 0000000000..d08eba52b8 --- /dev/null +++ b/npm/ng-packs/packages/theme-shared/src/lib/constants/index.ts @@ -0,0 +1,4 @@ +export * from './validation'; +export * from './default-errors'; +export * from './styles'; +export * from './scripts'; diff --git a/npm/ng-packs/packages/theme-shared/src/lib/services/create-error-component.service.ts b/npm/ng-packs/packages/theme-shared/src/lib/services/create-error-component.service.ts index da1ae0ddc0..92bafb840c 100644 --- a/npm/ng-packs/packages/theme-shared/src/lib/services/create-error-component.service.ts +++ b/npm/ng-packs/packages/theme-shared/src/lib/services/create-error-component.service.ts @@ -1,8 +1,9 @@ import { ApplicationRef, - ComponentFactoryResolver, ComponentRef, + createComponent, EmbeddedViewRef, + EnvironmentInjector, inject, Injectable, Injector, @@ -21,9 +22,9 @@ import { ErrorScreenErrorCodes } from '../models/common'; export class CreateErrorComponentService { protected readonly document = inject(DOCUMENT); protected readonly rendererFactory = inject(RendererFactory2); - protected readonly cfRes = inject(ComponentFactoryResolver); protected readonly routerEvents = inject(RouterEvents); - private readonly injector = inject(Injector); + protected readonly injector = inject(Injector); + protected readonly envInjector = inject(EnvironmentInjector); protected readonly httpErrorConfig = inject(HTTP_ERROR_CONFIG); componentRef: ComponentRef | null = null; @@ -65,9 +66,9 @@ export class CreateErrorComponentService { const hostElement = this.getErrorHostElement(); const host = renderer.selectRootElement(hostElement, true); - this.componentRef = this.cfRes - .resolveComponentFactory(HttpErrorWrapperComponent) - .create(this.injector); + this.componentRef = createComponent(HttpErrorWrapperComponent, { + environmentInjector: this.envInjector, + }); for (const key in instance) { /* istanbul ignore else */ @@ -80,9 +81,8 @@ export class CreateErrorComponentService { const appRef = this.injector.get(ApplicationRef); if (this.canCreateCustomError(instance.status as ErrorScreenErrorCodes)) { - this.componentRef.instance.cfRes = this.cfRes; this.componentRef.instance.appRef = appRef; - this.componentRef.instance.injector = this.injector; + this.componentRef.instance.environmentInjector = this.envInjector; this.componentRef.instance.customComponent = this.httpErrorConfig.errorScreen?.component; } @@ -91,6 +91,7 @@ export class CreateErrorComponentService { const destroy$ = new Subject(); this.componentRef.instance.destroy$ = destroy$; + destroy$.subscribe(() => { this.componentRef?.destroy(); this.componentRef = null; diff --git a/npm/ng-packs/packages/theme-shared/src/public-api.ts b/npm/ng-packs/packages/theme-shared/src/public-api.ts index 659e59e490..43411c916d 100644 --- a/npm/ng-packs/packages/theme-shared/src/public-api.ts +++ b/npm/ng-packs/packages/theme-shared/src/public-api.ts @@ -2,11 +2,9 @@ * Public API Surface of theme-shared */ -export * from './lib/adapters' +export * from './lib/adapters'; export * from './lib/animations'; export * from './lib/components'; -export * from './lib/constants/validation'; -export * from './lib/constants/default-errors'; export * from './lib/directives'; export * from './lib/enums'; export * from './lib/handlers'; @@ -16,3 +14,4 @@ export * from './lib/services'; export * from './lib/theme-shared.module'; export * from './lib/tokens'; export * from './lib/utils'; +export * from './lib/constants'; diff --git a/templates/app-nolayers/angular/src/app/app.component.ts b/templates/app-nolayers/angular/src/app/app.component.ts index a26e745334..44a504b978 100644 --- a/templates/app-nolayers/angular/src/app/app.component.ts +++ b/templates/app-nolayers/angular/src/app/app.component.ts @@ -5,6 +5,7 @@ import { Component } from '@angular/core'; template: ` + `, }) export class AppComponent {} diff --git a/templates/app-nolayers/angular/src/app/app.module.ts b/templates/app-nolayers/angular/src/app/app.module.ts index 13d8731b67..76e5c5ddcc 100644 --- a/templates/app-nolayers/angular/src/app/app.module.ts +++ b/templates/app-nolayers/angular/src/app/app.module.ts @@ -6,7 +6,7 @@ import { SettingManagementConfigModule } from '@abp/ng.setting-management/config import { TenantManagementConfigModule } from '@abp/ng.tenant-management/config'; import { ThemeLeptonXModule } from '@abp/ng.theme.lepton-x'; import { SideMenuLayoutModule } from '@abp/ng.theme.lepton-x/layouts'; -import { ThemeSharedModule } from '@abp/ng.theme.shared'; +import { InternetConnectionStatusComponent, ThemeSharedModule } from '@abp/ng.theme.shared'; import { NgModule } from '@angular/core'; import { BrowserModule } from '@angular/platform-browser'; import { BrowserAnimationsModule } from '@angular/platform-browser/animations'; @@ -35,6 +35,7 @@ import { AbpOAuthModule } from '@abp/ng.oauth'; ThemeLeptonXModule.forRoot(), SideMenuLayoutModule.forRoot(), FeatureManagementModule.forRoot(), + InternetConnectionStatusComponent ], declarations: [AppComponent], providers: [APP_ROUTE_PROVIDER], diff --git a/templates/app/angular/src/app/app.component.ts b/templates/app/angular/src/app/app.component.ts index a26e745334..44a504b978 100644 --- a/templates/app/angular/src/app/app.component.ts +++ b/templates/app/angular/src/app/app.component.ts @@ -5,6 +5,7 @@ import { Component } from '@angular/core'; template: ` + `, }) export class AppComponent {} diff --git a/templates/app/angular/src/app/app.module.ts b/templates/app/angular/src/app/app.module.ts index 34d59b4ec4..e09d7502fe 100644 --- a/templates/app/angular/src/app/app.module.ts +++ b/templates/app/angular/src/app/app.module.ts @@ -6,7 +6,7 @@ import { SettingManagementConfigModule } from '@abp/ng.setting-management/config import { TenantManagementConfigModule } from '@abp/ng.tenant-management/config'; import { ThemeLeptonXModule } from '@abp/ng.theme.lepton-x'; import { SideMenuLayoutModule } from '@abp/ng.theme.lepton-x/layouts'; -import { ThemeSharedModule } from '@abp/ng.theme.shared'; +import { InternetConnectionStatusComponent, ThemeSharedModule } from '@abp/ng.theme.shared'; import { NgModule } from '@angular/core'; import { BrowserModule } from '@angular/platform-browser'; import { BrowserAnimationsModule } from '@angular/platform-browser/animations'; @@ -36,6 +36,7 @@ import { AccountLayoutModule } from '@abp/ng.theme.lepton-x/account'; ThemeLeptonXModule.forRoot(), SideMenuLayoutModule.forRoot(), FeatureManagementModule.forRoot(), + InternetConnectionStatusComponent ], declarations: [AppComponent], providers: [APP_ROUTE_PROVIDER],