Skip to content

fix(oc-docs): dismiss playground overlay sidebar on outside click and fix dark-mode shadow - #132

Merged
bijin-bruno merged 2 commits into
opencollection-dev:mainfrom
sundram-bruno:fix/oc-docs-playground-sidebar-dismiss
Jul 22, 2026
Merged

fix(oc-docs): dismiss playground overlay sidebar on outside click and fix dark-mode shadow#132
bijin-bruno merged 2 commits into
opencollection-dev:mainfrom
sundram-bruno:fix/oc-docs-playground-sidebar-dismiss

Conversation

@sundram-bruno

@sundram-bruno sundram-bruno commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Close the inline-dock playground sidebar on outside click (backdrop), and replace its whitish dark-mode shadow with the theme-aware --oc-shadow-md token (also drops the doubled border and moves the docs drawer onto the same token).

JIRA : BRU-3753

… fix dark-mode shadow

The inline-dock sidebar overlays the view but only closed on navigation,
so a click outside it did nothing. Add a backdrop that closes it, mirroring
the docs navigation drawer.

Also swap the overlay's text-tinted color-mix shadow (which turned whitish
in dark mode) for the theme-aware --oc-shadow-md token, drop the doubled
border-right in overlay mode, and move the docs drawer onto the same token.
@sundram-bruno
sundram-bruno force-pushed the fix/oc-docs-playground-sidebar-dismiss branch from cc154c3 to 516cfc1 Compare July 21, 2026 09:39
Comment thread packages/oc-docs/src/components/Playground/PlaygroundBody/StyledWrapper.ts Outdated
…for playground overlay

Add useClickOutside so the inline-dock sidebar also closes when clicking
outside the playground, alongside the backdrop that blocks clicks over the
view. Extend the hook with an optional ignoreSelector so the sidebar toggle
is not treated as outside (which would reopen it). Replace the hardcoded
overlay z-indexes with the --z-sidebar token.
@bijin-bruno
bijin-bruno merged commit c6a917b into opencollection-dev:main Jul 22, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants