Addresses code review feedback from PR #94: duplicate layout components violating DRY, memory leaks from unsubscribed observables, and missing keyboard accessibility handlers.

Changes

• Removed duplicate layout components: HomeLayoutComponent, MovementsLayoutComponent, and AnalyticsLayoutComponent were identical wrappers containing only <router-outlet />. Routes now reference child components directly.

• Fixed memory leaks: Router event subscriptions in BreadcrumbComponent and SideNavComponent now use takeUntilDestroyed() from @angular/core/rxjs-interop for automatic cleanup.

• Added keyboard accessibility: Theme toggle and logout buttons now handle keydown.enter and keydown.space events with preventDefault() to support keyboard navigation without double-triggering.

Before (routes.ts):

{
  path: 'home',
  component: HomeLayoutComponent,  // Unnecessary wrapper
  children: [...]
}

After:

{
  path: 'home',
  children: [...]  // Direct child routes
}

Before (subscriptions):

ngOnInit(): void {
  this.router.events
    .pipe(filter(...))
    .subscribe(...);  // Never unsubscribed
}

After:

constructor() {
  this.router.events
    .pipe(filter(...), takeUntilDestroyed())
    .subscribe(...);
}

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.