feat: add quick label selection mode for one-click annotations#272
feat: add quick label selection mode for one-click annotations#272backnotprop merged 6 commits intobacknotprop:mainfrom
Conversation
Add preset label chips (Needs tests, Security concern, Break this up, etc.) that allow instant annotation without typing. Includes ⚡ toolbar button, Alt+1..8 keyboard shortcuts, label customization in Settings, and label summary in export output. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
@grubmanItay really enjoying this feature. Great job. I'm changing up some of the quick label orders. I'm happy to meet with you to discuss. We're also going to start a discussion. |
- Redesign FloatingQuickLabelPicker as a vertical context-menu style list with cursor-anchored positioning (appears at mouseup point, not selection center) - Unify label dropdown: toolbar and quick-label mode now share the same FloatingQuickLabelPicker component (removed duplicate InlineQuickLabelDropdown) - Fix above/below flip positioning (follow CommentPopover pattern with conditional translateY) - Add label tips: optional instruction text on QuickLabel that gets injected into agent feedback as a blockquote below the label - Add tip editor in Settings with three visual states (empty/editing/filled) - Add "Missing overview" default label with a tip for requesting narrative context - Extend keyboard shortcuts from Alt+1-8 to Alt+1-9 - Suppress input method toggle (Alt) when label picker is open - Reorder default labels: Clarify this, Needs tests, Consider edge cases, Missing overview, Security concern, Break this up, Wrong order, Discuss first, Nice approach Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
I'm gonna fix all these merge conflicts by the way. |
Finalize the 10 default quick labels based on user feedback data: clarify, overview, verify, example, patterns, alternatives, regression, out-of-scope, tests, nice-approach. Each label gets a unique color (added cyan and amber to the palette). Bare digit keys (1-0) now apply labels when the picker is open, Alt+N still works everywhere. Tip editor cursor starts at beginning for readability. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
# Conflicts: # bun.lock # packages/ui/components/AnnotationToolbar.tsx # packages/ui/components/KeyboardShortcuts.tsx # packages/ui/components/Settings.tsx # packages/ui/components/Viewer.tsx # packages/ui/types.ts # packages/ui/utils/parser.ts # packages/ui/utils/quickLabels.ts
Code reviewFound 1 issue:
🤖 Generated with Claude Code - If this code review was useful, please react with 👍. Otherwise, react with 👎. |
…nd picker handlers When the quick label picker is open from the toolbar's zap button, let FloatingQuickLabelPicker own all keyboard input instead of both components handling the same keypress. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Description
Follow-up to #268 — implements the "quick label selection mode" suggested by @backnotprop.
⚡ New Mode: Quick Label
Adds a 4th editor mode alongside Markup, Comment, and Redline. When active:
Works with both input methods (Select and Pinpoint), including code blocks.
How It Works
Mode persists across sessions via cookie storage (same as other modes).
Files Changed
packages/ui/types.ts'quickLabel'toEditorModeunionpackages/ui/utils/editorMode.tspackages/ui/components/QuickLabelDropdown.tsxpackages/ui/components/FloatingQuickLabelPicker.tsxpackages/ui/components/AnnotationToolbar.tsxQuickLabelDropdownpackages/ui/components/Viewer.tsxquickLabelbranch in CREATE handler + pinpoint handler + state managementpackages/ui/components/AnnotationToolstrip.tsxwarningcolor entrypackages/ui/components/KeyboardShortcuts.tsx