diff --git a/ui/src/app/app.routing.ts b/ui/src/app/app.routing.ts index 34ad45010..091cec57b 100644 --- a/ui/src/app/app.routing.ts +++ b/ui/src/app/app.routing.ts @@ -16,7 +16,9 @@ const routes: Routes = [ @NgModule({ imports: [RouterModule.forRoot(routes, { preloadingStrategy: PreloadAllModules, - scrollOffset: [0, 64] + scrollOffset: [0, 64], + anchorScrolling: 'enabled', + scrollPositionRestoration: 'enabled' })], exports: [RouterModule] }) diff --git a/ui/src/app/metadata/configuration/container/metadata-options.component.html b/ui/src/app/metadata/configuration/container/metadata-options.component.html index bcb699b73..54c969b5e 100644 --- a/ui/src/app/metadata/configuration/container/metadata-options.component.html +++ b/ui/src/app/metadata/configuration/container/metadata-options.component.html @@ -44,24 +44,26 @@

Loading... -
-
-

- Filters -

-
- -   - Add Filter - +
+ +
+

+ Filters +

+
-
- + +
  diff --git a/ui/src/app/metadata/provider/container/provider-edit.component.spec.ts b/ui/src/app/metadata/provider/container/provider-edit.component.spec.ts index a5601d594..81da40d38 100644 --- a/ui/src/app/metadata/provider/container/provider-edit.component.spec.ts +++ b/ui/src/app/metadata/provider/container/provider-edit.component.spec.ts @@ -104,10 +104,10 @@ describe('Provider Edit Component', () => { }); describe('cancel method', () => { - it('should route to the metadata manager', () => { + it('should route to the provider', () => { spyOn(router, 'navigate'); spyOn(app, 'clear'); - app.cancel(); + app.cancel('foo'); expect(router.navigate).toHaveBeenCalled(); expect(app.clear).toHaveBeenCalled(); }); diff --git a/ui/src/app/metadata/provider/container/provider-edit.component.ts b/ui/src/app/metadata/provider/container/provider-edit.component.ts index c00da63e2..ef37af038 100644 --- a/ui/src/app/metadata/provider/container/provider-edit.component.ts +++ b/ui/src/app/metadata/provider/container/provider-edit.component.ts @@ -90,9 +90,9 @@ export class ProviderEditComponent implements OnDestroy, CanComponentDeactivate this.store.dispatch(new UpdateProviderRequest(this.latest)); } - cancel(): void { + cancel(id): void { this.clear(); - this.router.navigate(['dashboard', 'metadata', 'manager', 'providers']); + this.router.navigate(['/', 'metadata', 'provider', id, 'configuration', 'options']); } canDeactivate( diff --git a/ui/src/app/metadata/provider/container/provider-filter-list.component.html b/ui/src/app/metadata/provider/container/provider-filter-list.component.html index 253674e20..f38e66d7b 100644 --- a/ui/src/app/metadata/provider/container/provider-filter-list.component.html +++ b/ui/src/app/metadata/provider/container/provider-filter-list.component.html @@ -24,7 +24,12 @@
-
+
+ +   + Return to Provider + diff --git a/ui/src/app/metadata/provider/effect/collection.effect.ts b/ui/src/app/metadata/provider/effect/collection.effect.ts index 8b1003912..f319a3d20 100644 --- a/ui/src/app/metadata/provider/effect/collection.effect.ts +++ b/ui/src/app/metadata/provider/effect/collection.effect.ts @@ -136,7 +136,7 @@ export class CollectionEffects { createProviderSuccessRedirect$ = this.actions$.pipe( ofType(ProviderCollectionActionTypes.ADD_PROVIDER_SUCCESS), map(action => action.payload), - tap(provider => this.router.navigate(['dashboard', 'metadata', 'manager', 'providers'])) + tap(provider => this.navigateToProvider(provider.resourceId)) ); @Effect() @@ -149,7 +149,7 @@ export class CollectionEffects { this.providerService .update(provider) .pipe( - map(p => new UpdateProviderSuccess({id: p.id, changes: p})), + map(p => new UpdateProviderSuccess({id: p.resourceId, changes: p})), catchError((e) => e.status === 409 ? of(new UpdateProviderConflict(provider)) : of(new UpdateProviderFail(e.error))) ) ) @@ -168,7 +168,7 @@ export class CollectionEffects { map(action => action.payload), tap(provider => { this.store.dispatch(new ClearProvider()); - this.router.navigate(['dashboard', 'metadata', 'manager', 'providers']); + this.navigateToProvider(provider.id); }) ); @@ -269,4 +269,8 @@ export class CollectionEffects { private contentionService: ContentionService, private i18nService: I18nService ) { } + + navigateToProvider(id) { + this.router.navigate(['/', 'metadata', 'provider', id, 'configuration', 'options']); + } }