Skip to content

Commit dda32dd

Browse files
committed
Implement community feedback p2
1 parent 51127dc commit dda32dd

14 files changed

Lines changed: 389 additions & 108 deletions

src/app/app.menus.ts

Lines changed: 12 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ import { ProcessesMenuProvider } from './shared/menu/providers/processes.menu';
3131
import { RegistriesMenuProvider } from './shared/menu/providers/registries.menu';
3232
import { StatisticsMenuProvider } from './shared/menu/providers/statistics.menu';
3333
import { SystemWideAlertMenuProvider } from './shared/menu/providers/system-wide-alert.menu';
34+
import { WithdrawnReinstateItemMenuProvider } from './shared/menu/providers/withdrawn-reinstate-item.menu';
3435
import { WorkflowMenuProvider } from './shared/menu/providers/workflow.menu';
3536

3637
/**
@@ -75,32 +76,25 @@ export const MENUS = buildMenuStructure({
7576
[MenuID.DSO_EDIT]: [
7677
DsoOptionMenuProvider.withSubs([
7778
SubscribeMenuProvider.onRoute(
78-
MenuRoute.SIMPLE_COMMUNITY_PAGE,
79-
MenuRoute.SIMPLE_COMMUNITY_BROWSE_PAGE,
80-
MenuRoute.SIMPLE_COMMUNITY_SUBCOLCOM_PAGE,
81-
MenuRoute.SIMPLE_COLLECTION_PAGE,
82-
MenuRoute.SIMPLE_COLLECTION_BROWSE_PAGE,
79+
MenuRoute.COMMUNITY_PAGE,
80+
MenuRoute.COLLECTION_PAGE,
8381
),
8482
DSpaceObjectEditMenuProvider.onRoute(
85-
MenuRoute.SIMPLE_COMMUNITY_PAGE,
86-
MenuRoute.SIMPLE_COMMUNITY_BROWSE_PAGE,
87-
MenuRoute.SIMPLE_COMMUNITY_SUBCOLCOM_PAGE,
88-
MenuRoute.SIMPLE_COLLECTION_PAGE,
89-
MenuRoute.SIMPLE_COLLECTION_BROWSE_PAGE,
90-
MenuRoute.SIMPLE_ITEM_PAGE,
91-
MenuRoute.FULL_ITEM_PAGE,
83+
MenuRoute.COMMUNITY_PAGE,
84+
MenuRoute.COLLECTION_PAGE,
85+
MenuRoute.ITEM_PAGE,
86+
),
87+
WithdrawnReinstateItemMenuProvider.onRoute(
88+
MenuRoute.ITEM_PAGE,
9289
),
9390
VersioningMenuProvider.onRoute(
94-
MenuRoute.SIMPLE_ITEM_PAGE,
95-
MenuRoute.FULL_ITEM_PAGE,
91+
MenuRoute.ITEM_PAGE,
9692
),
9793
OrcidMenuProvider.onRoute(
98-
MenuRoute.SIMPLE_ITEM_PAGE,
99-
MenuRoute.FULL_ITEM_PAGE,
94+
MenuRoute.ITEM_PAGE,
10095
),
10196
ClaimMenuProvider.onRoute(
102-
MenuRoute.SIMPLE_ITEM_PAGE,
103-
MenuRoute.FULL_ITEM_PAGE,
97+
MenuRoute.ITEM_PAGE,
10498
),
10599
]),
106100
],

src/app/collection-page/collection-page-routes.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ export const ROUTES: Route[] = [
8181
path: '',
8282
component: ThemedCollectionPageComponent,
8383
data: {
84-
menuRoute: MenuRoute.SIMPLE_COLLECTION_PAGE,
84+
menuRoute: MenuRoute.COLLECTION_PAGE,
8585
},
8686
children: [
8787
{
@@ -99,7 +99,7 @@ export const ROUTES: Route[] = [
9999
},
100100
data: {
101101
breadcrumbKey: 'browse.metadata',
102-
menuRoute: MenuRoute.SIMPLE_COLLECTION_BROWSE_PAGE,
102+
menuRoute: MenuRoute.COLLECTION_PAGE,
103103
},
104104
},
105105
],

src/app/community-page/community-page-routes.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ export const ROUTES: Route[] = [
6868
path: '',
6969
component: ThemedCommunityPageComponent,
7070
data: {
71-
menuRoute: MenuRoute.SIMPLE_COMMUNITY_PAGE,
71+
menuRoute: MenuRoute.COMMUNITY_PAGE,
7272
},
7373
children: [
7474
{
@@ -85,7 +85,7 @@ export const ROUTES: Route[] = [
8585
},
8686
data: {
8787
breadcrumbKey: 'community.subcoms-cols',
88-
menuRoute: MenuRoute.SIMPLE_COMMUNITY_SUBCOLCOM_PAGE,
88+
menuRoute: MenuRoute.COMMUNITY_PAGE,
8989
},
9090
},
9191
{
@@ -98,7 +98,7 @@ export const ROUTES: Route[] = [
9898
},
9999
data: {
100100
breadcrumbKey: 'browse.metadata',
101-
menuRoute: MenuRoute.SIMPLE_COMMUNITY_BROWSE_PAGE,
101+
menuRoute: MenuRoute.COMMUNITY_PAGE,
102102
},
103103
},
104104
],

src/app/init.service.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,6 @@ export abstract class InitService {
217217
this.headTagService.listenForRouteChange();
218218
this.breadcrumbsService.listenForRouteChanges();
219219
this.themeService.listenForRouteChanges();
220-
this.menuProviderService.listenForRouteChanges();
221220
}
222221

223222
/**

src/app/item-page/item-page-routes.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,15 +33,15 @@ export const ROUTES: Route[] = [
3333
component: ThemedItemPageComponent,
3434
pathMatch: 'full',
3535
data: {
36-
menuRoute: MenuRoute.SIMPLE_ITEM_PAGE,
36+
menuRoute: MenuRoute.ITEM_PAGE,
3737
},
3838

3939
},
4040
{
4141
path: 'full',
4242
component: ThemedFullItemPageComponent,
4343
data: {
44-
menuRoute: MenuRoute.FULL_ITEM_PAGE,
44+
menuRoute: MenuRoute.ITEM_PAGE,
4545
},
4646

4747
},

src/app/shared/menu/menu-provider.model.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,11 @@ export abstract class AbstractMenuProvider implements MenuProvider {
111111
*/
112112
alwaysRenderExpandable? = false;
113113

114+
/**
115+
* When true, this provider will only add its sections on Browser Side Rendering.
116+
*/
117+
renderBrowserOnly? = false;
118+
114119
/**
115120
* Static method to be called from the app.menus.ts file to define paths on which this provider should the active
116121
* @param paths - The paths on which the sections of this provider should be active

0 commit comments

Comments
 (0)