Skip to content

fix: Recent Documents sidebar can be closed from its header - #21

Merged
b451c merged 1 commit into
b451c:mainfrom
Coriou:ui/recent-docs-header-collapse
Aug 28, 2026
Merged

fix: Recent Documents sidebar can be closed from its header#21
b451c merged 1 commit into
b451c:mainfrom
Coriou:ui/recent-docs-header-collapse

Conversation

@Coriou

@Coriou Coriou commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Me again with a half-baked, visually untested PR.

Once the Recent Documents sidebar is open, there is no in-panel way to close it. Shift-Cmd-D still toggles it; this just adds a collapse control on the Documents header (same icon as the floating capsule, inverse verb).

Still no Xcode on this machine, so I have not seen it run. The button clones the existing trash control. MarkdownView still owns the flag.

Two more PRs coming on the same theme:

  • the same collapse control on the Table of Contents header
  • moving Table of Contents and Recent Documents from Edit to View

If you can't be bothered to clean up after me, I'll take care of it when I get home.

The sidebar could be opened from the floating capsule but had no in-panel
close. Header collapse clones the trash button (sidebar.leading, inverse
of the overlay); MarkdownView still owns the flag; ⇧⌘D still toggles.
@b451c
b451c merged commit 48124b1 into b451c:main Aug 28, 2026
3 checks passed
b451c added a commit that referenced this pull request Aug 28, 2026
… single-line Documents title

- CHANGELOG: the three PRs each opened their own [Unreleased]; merged into one with credits (also the Homebrew cask fix, #25/#26 + tap #2).
- View menu: divider between the sidebar toggles and Reading Mode.
- Recent Documents header: with two buttons in the row the title wrapped to two lines at the 160 pt minimum width; lineLimit(1) + fixedSize keeps it on one.

Verified: 186 tests, Debug + Release + Release-APPSTORE build; AX-driven E2E on the Debug build (both collapse buttons, View/Edit menu contents, flags untouched by Reading Mode, 160 pt sidebar).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HHNj7eLDeEN5RDQXSZdRco
@b451c

b451c commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Merged - thank you! Review notes for all three PRs are in #23. One follow-up on main: with two buttons in the header row the "Documents" title wrapped at the 160 pt minimum sidebar width, so it now has lineLimit(1) + fixedSize.

b451c added a commit that referenced this pull request Aug 28, 2026
Sidebar collapse controls and View-menu move (#21, #22, #23 @Coriou), Homebrew cask depends_on fix (#25, #26 @robin-moser; tap #2 @GregOriol).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HHNj7eLDeEN5RDQXSZdRco
@b451c

b451c commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Shipped in v1.9.1: https://github.com/b451c/quickmd/releases/tag/v1.9.1 (GitHub + Homebrew live; App Store build submitted). Thanks again!

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.

2 participants