From 8ba58161ac59a29d58244f48dc9dbff466b1e82a Mon Sep 17 00:00:00 2001 From: Juraj Roka <95219754+jr-rk@users.noreply.github.com> Date: Tue, 30 Jun 2026 15:33:52 +0200 Subject: [PATCH] Refactor: align eager-themes.module.ts with the other customer backports Rebuilds this follow-up on the current customer/vsb-tuo base (which now carries the final overlay mechanism #1318/#1321 and #1333). Drops the duplicate dspace eager-theme import (imported once as DSpaceEagerThemeModule and again, unaliased, as EagerThemeModule) so this file matches the canonical form on TUL/SAV/ZCU-DATA/ ZCU-PUB: dspace eager theme once + custom eager theme once. The earlier blank-page guard from this branch is intentionally dropped: #1321 removed the ds-app visibility:hidden rule (the overlay is now a purely visual mask), so the early-return can no longer hide the app, and the other customers took VSB-TUO's #1321 index.html verbatim. Keeping a guard only here would re-diverge index.html. Co-Authored-By: Claude Opus 4.8 --- src/themes/eager-themes.module.ts | 5 ----- 1 file changed, 5 deletions(-) diff --git a/src/themes/eager-themes.module.ts b/src/themes/eager-themes.module.ts index 93518ee01a8..29d46032de8 100644 --- a/src/themes/eager-themes.module.ts +++ b/src/themes/eager-themes.module.ts @@ -1,6 +1,5 @@ import { NgModule } from '@angular/core'; import { EagerThemeModule as DSpaceEagerThemeModule } from './dspace/eager-theme.module'; -import { EagerThemeModule } from './dspace/eager-theme.module'; import { EagerThemeModule as CustomEagerThemeModule } from './custom/eager-theme.module'; /** @@ -18,10 +17,6 @@ import { EagerThemeModule as CustomEagerThemeModule } from './custom/eager-theme @NgModule({ imports: [ DSpaceEagerThemeModule, - // Uncomment this because the `untyped-item` theming is not working when it is commented out. - // Issue: https://github.com/DSpace/dspace-angular/issues/1897 - // Useful info in PR: https://github.com/DSpace/dspace-angular/pull/2262#issuecomment-1557146081 - EagerThemeModule, CustomEagerThemeModule, ], })