diff --git a/src/apps/officer-web/Screens/Dashboard/Layout/DashboardLayout.tsx b/src/apps/officer-web/Screens/Dashboard/Layout/DashboardLayout.tsx index b3491b8d..60f27099 100644 --- a/src/apps/officer-web/Screens/Dashboard/Layout/DashboardLayout.tsx +++ b/src/apps/officer-web/Screens/Dashboard/Layout/DashboardLayout.tsx @@ -8,6 +8,7 @@ import { Background } from './Background'; import { Header } from './Header'; import { Dock, CORE_DOCK_ITEMS, dockItemsFromPlugins, defaultDockPaths } from './Dock'; import { Spotlight } from './Spotlight/Spotlight'; +import { ShortcutsModal } from './ShortcutsModal'; import { useIsTouch } from './useIsTouch'; import { usePageTitleSync } from '@/state/usePageTitle'; import { RouteGate } from './RouteGate'; @@ -51,6 +52,9 @@ export function DashboardLayout({ children }: DashboardLayoutProps) { {/* Outside the content region on purpose, like the header and the dock: whatever a screen is doing — maximised panel, error boundary, mid-navigation — ctrl-space still gets you off it. */} + {/* Outside the content region for the same reason Spotlight is: ctrl+? has to work from a maximised + panel, an error boundary, or mid-navigation. */} + {/* overflow-CLIP, not hidden: `hidden` still makes this a scroll container, and the nav Dock — absolute, parked below the bottom edge by translateY while hidden — adds its transformed box to the scrollable overflow. So clicking a link let the browser scroll this section ~70px to reveal diff --git a/src/apps/officer-web/Screens/Dashboard/Layout/Header/ThemeSwitcher.tsx b/src/apps/officer-web/Screens/Dashboard/Layout/Header/ThemeSwitcher.tsx index 0c1c7b86..1a1694e8 100644 --- a/src/apps/officer-web/Screens/Dashboard/Layout/Header/ThemeSwitcher.tsx +++ b/src/apps/officer-web/Screens/Dashboard/Layout/Header/ThemeSwitcher.tsx @@ -23,7 +23,7 @@ const { * * ── The overlay ── * - * `ctrl+alt+space` opens a list in the middle of the screen; `j`/`k` move through it and `l` applies the + * `ctrl+shift+space` opens a list in the middle of the screen; `j`/`k` move through it and `l` applies the * focused theme to the app BEHIND it. That is the whole idea, and it is why this is a portal over a * translucent backdrop rather than a modal dialog: you are meant to watch the application change while * you browse, so as little of it as possible should be covered. @@ -90,7 +90,7 @@ export const ThemeSwitcher = () => {
ctrl+shift+←/→ cycle · ctrl+shift+space browse - alt+space to search every theme by name + ctrl+? for every shortcut