Skip to content

feat(documentation-dialog): collapsible sections for docs content - #95

Merged
alpha5611331 merged 2 commits into
mainfrom
feat/collapsible-documentation-dialog
Aug 10, 2026
Merged

feat(documentation-dialog): collapsible sections for docs content#95
alpha5611331 merged 2 commits into
mainfrom
feat/collapsible-documentation-dialog

Conversation

@alpha5611331

Copy link
Copy Markdown
Member

Summary

  • Added shadcn Accordion component (src/renderer/components/ui/accordion.tsx)
  • Refactored DocumentationDialog so "Lost the window?" and "Hotkeys" render as collapsible accordion sections instead of always-expanded text blocks, for a cleaner, more professional and scannable dialog
  • "Hotkeys" is expanded by default since it's the most-referenced section; "Lost the window?" starts collapsed

Closes #94

Test plan

  • pnpm exec tsc -b — passes
  • pnpm exec eslint on changed files — passes
  • pnpm build — production build succeeds
  • Manual click-through in the running app (not exercised in this sandbox — no display/Playwright available here; please verify visually before merge)

Needed for collapsible sections in the documentation dialog (#94).
Replaces always-expanded "Lost the window?" and "Hotkeys" blocks with
an accordion so the dialog reads cleaner and scans faster. Hotkeys
stays open by default since it's the most-referenced section.

Closes #94
@gitar-bot

gitar-bot Bot commented Aug 10, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Adds collapsible accordion sections to the documentation dialog using a new shadcn component, improving content scannability. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@alpha5611331
alpha5611331 merged commit 8277b99 into main Aug 10, 2026
2 checks passed
@alpha5611331
alpha5611331 deleted the feat/collapsible-documentation-dialog branch August 10, 2026 18:40

@anton-karlovskiy anton-karlovskiy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@chmm195 chmm195 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@kevinkamto

Copy link
Copy Markdown
Collaborator

Good job 👍

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.

Documentation dialog: collapsible sections for better UI/UX

4 participants