Skip to content

Migrate sidebar to @posthog/quill#1663

Merged
adamleithp merged 10 commits intomainfrom
quill/sidenav
Apr 15, 2026
Merged

Migrate sidebar to @posthog/quill#1663
adamleithp merged 10 commits intomainfrom
quill/sidenav

Conversation

@adamleithp
Copy link
Copy Markdown
Contributor

@adamleithp adamleithp commented Apr 15, 2026

Summary

  • Sidebar components (items, sections, menu, scroll area, task filter) swapped onto @posthog/quill primitives; deletes the old quill.css shim.
  • Installs @posthog/quill@0.1.0-alpha.6 from npm (replacing the local link: install) and wires tokens.css / base.css / tailwind.css into globals.css per quill's shadcn-style install docs.
  • TaskFilterMenu rebuilt as a quill DropdownMenu with radio selection + align-start so the active filter is visible.

Before:

image image
2026-04-15 10 14 50
2026-04-15 10 15 46

Test plan

  • pnpm --filter code typecheck
  • pnpm --filter code test
  • Launch app, sidebar renders in light + dark, scroll works, section collapse/expand works
  • Task filter dropdown opens, radio reflects current filter, selection updates list
  • Sidebar item active/hover states still correct across Home, Skills, Tasks, Projects

🤖 Generated with Claude Code

Comment thread apps/code/package.json
"rehype-sanitize": "^6.0.0",
"remark-breaks": "^4.0.0",
"remark-gfm": "^4.0.1",
"shadcn": "^4.1.2",
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.

would it make sense to put this into the pnpm catalog?

@@ -1,161 +1,161 @@
import { Command as CmdkCommand } from "cmdk";
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.

is there a reason we dont just use the file from Quill instead?

(Is there even a Command in quill)

Copy link
Copy Markdown
Contributor

@jonathanlab jonathanlab left a comment

Choose a reason for hiding this comment

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

<3

Copy link
Copy Markdown
Member

@charlesvien charlesvien left a comment

Choose a reason for hiding this comment

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

LGTM

@adamleithp adamleithp enabled auto-merge (squash) April 15, 2026 20:41
@adamleithp adamleithp merged commit 0fd4584 into main Apr 15, 2026
15 checks passed
@adamleithp adamleithp deleted the quill/sidenav branch April 15, 2026 20:43
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.

4 participants