1 Commits
Author SHA1 Message Date
TZGyn c7b4ed32f2 Make the row checkbox select items instead of ticking them off
The checkbox was marking items done, but the intent was to gather items
and act on them: tick several, then right-click to copy, copy as a list,
move, merge or delete them all at once. Done is still available from the
same menu, it is just no longer what the checkbox does.
2026-10-09 17:54:54 +08:00
4 changed files with 96 additions and 44 deletions
+13 -7
View File
@@ -27,13 +27,17 @@ clicking an item copies it back to the clipboard.
`Images/` next to it, cached in an `NSCache`. Owns all mutations (add, `Images/` next to it, cached in an `NSCache`. Owns all mutations (add,
toggle done, edit, delete, move, merge, clear done) and clipboard writes toggle done, edit, delete, move, merge, clear done) and clipboard writes
(`copy`, `copyAsList` for numbered output, `copyAsLines` for plain newlines; (`copy`, `copyAsList` for numbered output, `copyAsLines` for plain newlines;
both text modes skip image items). Saves on every mutation. both text modes skip image items). Batch mutations (`delete(ids:)`,
`setDone(_:ids:)`, `move(ids:to:)`) back the selection context menu, so one
action applies to every selected item and image files are cleaned up once.
Saves on every mutation.
- `Sources/Prompt/PanelView.swift` — the panel UI. Header (section switcher, - `Sources/Prompt/PanelView.swift` — the panel UI. Header (section switcher,
search, add photos, close), item list, composer. Composer `# Name` creates/ search, add photos, close), item list, composer. Composer `# Name` creates/
switches sections. Click = copy; Cmd+click toggles multi-select, Shift+click switches sections. Click = copy; the row checkbox selects, Cmd+click toggles
selects a range, and a selection bar offers Copy as List / Copy as Lines; multi-select, Shift+click selects a range, and a selection bar offers Copy as
right-click = context menu (Copy, Copy as List, Copy as Lines, Done, Edit, List / Copy as Lines; right-click = context menu acting on the selection
Open, Merge, Move To, Delete). Esc clears the selection before it hides the (Copy, Copy as List, Copy as Lines, Done, Edit, Open, Merge, Move To,
Delete). Esc clears the selection before it hides the
panel. Drop target for images/files. panel. Drop target for images/files.
- `Sources/Prompt/PanelController.swift` — owns the floating `NSPanel` - `Sources/Prompt/PanelController.swift` — owns the floating `NSPanel`
(`.nonactivatingPanel`, `.floating`, right-edge docked, frame autosaved as (`.nonactivatingPanel`, `.floating`, right-edge docked, frame autosaved as
@@ -169,8 +173,10 @@ brew upgrade --cask prompt
clipboard image to the store when there is no text. `Prompt --diag` dumps the clipboard image to the store when there is no text. `Prompt --diag` dumps the
menu so the wiring can be checked without a keyboard. menu so the wiring can be checked without a keyboard.
- **A row is not a `Button`** because it contains a checkbox `Button` for - **A row is not a `Button`** because it contains a checkbox `Button` for
ticking items off, and nested buttons never receive clicks. The row uses selecting the item, and nested buttons never receive clicks. The row uses
`contentShape` + `onTapGesture` for click-to-copy instead. `contentShape` + `onTapGesture` for click-to-copy instead. Selection is
driven by the checkbox, Cmd+click (toggle) and Shift+click (range); the
context menu acts on the whole selection via `effectiveIDs(for:)`.
- **Never bind bare keys as `.keyboardShortcut` in the panel** (Space, Delete, - **Never bind bare keys as `.keyboardShortcut` in the panel** (Space, Delete,
plain Cmd+C). SwiftUI key equivalents are matched before text input and plain Cmd+C). SwiftUI key equivalents are matched before text input and
would steal characters from the composer TextField. Only Cmd-modified would steal characters from the composer TextField. Only Cmd-modified
+8 -5
View File
@@ -4,8 +4,9 @@ A native macOS app for keeping prompts and photos, in the spirit of
[Copper](https://shadcn.com/copper) by shadcn: a floating panel that sits next [Copper](https://shadcn.com/copper) by shadcn: a floating panel that sits next
to your work. Click anything to copy it back to your clipboard. to your work. Click anything to copy it back to your clipboard.
- **Checklists by default** — every item has a checkbox, so a workspace doubles - **Checkbox to select** — every item has a checkbox; tick several and act on
as a to-do list; done items stay in place, dimmed and struck through. them at once (copy, delete, move, merge). Clicking the row itself still
copies that item.
- **Click to copy** — text notes go back as text, photos go back as images. - **Click to copy** — text notes go back as text, photos go back as images.
- **Capture from anywhere** — select text in any app and tap **Left Shift twice**. - **Capture from anywhere** — select text in any app and tap **Left Shift twice**.
- **Photos** — paste (Cmd+V), drag & drop, or add via the photo button. - **Photos** — paste (Cmd+V), drag & drop, or add via the photo button.
@@ -13,9 +14,11 @@ to your work. Click anything to copy it back to your clipboard.
the Cmd+K switcher (which opens a name modal); Cmd+K switches, and the the Cmd+K switcher (which opens a name modal); Cmd+K switches, and the
up/down arrows in the switcher reorder them. (The code and the saved JSON up/down arrows in the switcher reorder them. (The code and the saved JSON
still call these `sections` so existing data keeps working.) still call these `sections` so existing data keeps working.)
- **Copy as List / Copy as Lines** — Cmd+click to multi-select (Shift+click for - **Copy as List / Copy as Lines** — select with the checkboxes (or Cmd+click,
a range), then copy the selection as a numbered list or as plain or Shift+click for a range), then copy the selection as a numbered list or as
newline-separated text, from the selection bar or the right-click menu. plain newline-separated text, from the selection bar, the right-click menu, or
Shift+Cmd+C / Option+Cmd+C. The same menu deletes, moves or merges the
selection.
- **Merge Notes, Mark as Done, Edit, Move To** — right-click any item. - **Merge Notes, Mark as Done, Edit, Move To** — right-click any item.
- **Search** — Cmd+F searches across all sections. - **Search** — Cmd+F searches across all sections.
- **Sync (optional)** — generate an 8-digit code (`1234-5678`) on one Mac and - **Sync (optional)** — generate an 8-digit code (`1234-5678`) on one Mac and
+39 -27
View File
@@ -193,20 +193,20 @@ struct PanelView: View {
.padding(.vertical, 48) .padding(.vertical, 48)
} }
/// Rows are checklists by default: a checkbox to tick an item off, with the /// Rows carry a checkbox that *selects* the item, so a selection can be
/// rest of the row copying on click. The row cannot be a Button because the /// copied or deleted from the context menu. The row cannot be a Button
/// checkbox is one, and nested buttons do not receive clicks. /// because the checkbox is one, and nested buttons do not receive clicks.
private func row(for item: PromptItem, sectionName: String) -> some View { private func row(for item: PromptItem, sectionName: String) -> some View {
HStack(alignment: .top, spacing: 8) { HStack(alignment: .top, spacing: 8) {
Button { Button {
store.toggleDone(item.id) toggleSelection(item.id)
} label: { } label: {
Image(systemName: item.done ? "checkmark.circle.fill" : "circle") Image(systemName: selection.contains(item.id) ? "checkmark.circle.fill" : "circle")
.font(.system(size: 15)) .font(.system(size: 15))
.foregroundStyle(item.done ? Color.accentColor : Color.secondary.opacity(0.55)) .foregroundStyle(selection.contains(item.id) ? Color.accentColor : Color.secondary.opacity(0.55))
} }
.buttonStyle(.plain) .buttonStyle(.plain)
.help(item.done ? "Mark as not done" : "Mark as done") .help(selection.contains(item.id) ? "Deselect" : "Select for copy or delete")
VStack(alignment: .leading, spacing: 6) { VStack(alignment: .leading, spacing: 6) {
if !sectionName.isEmpty { if !sectionName.isEmpty {
@@ -278,43 +278,51 @@ struct PanelView: View {
} }
private func contextMenu(for item: PromptItem) -> some View { private func contextMenu(for item: PromptItem) -> some View {
Group { let ids = effectiveIDs(for: item)
let multiple = ids.count > 1
return Group {
Button("Copy") { copy(item) } Button("Copy") { copy(item) }
Button("Copy as List") { Button(multiple ? "Copy \(ids.count) as List" : "Copy as List") {
store.copyAsList(ids: effectiveIDs(for: item)) store.copyAsList(ids: ids)
flashCopied(item.id) flashCopied(item.id)
} }
Button("Copy as Lines") { Button(multiple ? "Copy \(ids.count) as Lines" : "Copy as Lines") {
store.copyAsLines(ids: effectiveIDs(for: item)) store.copyAsLines(ids: ids)
flashCopied(item.id) flashCopied(item.id)
} }
Divider() Divider()
Button(item.done ? "Mark as Not Done" : "Mark as Done") { store.toggleDone(item.id) } Button(item.done ? "Mark as Not Done" : "Mark as Done") {
if case .text(let text) = item.content { store.setDone(!item.done, ids: ids)
}
if case .text(let text) = item.content, !multiple {
Button("Edit…") { Button("Edit…") {
PanelController.shared.editController.edit(itemID: item.id, text: text) PanelController.shared.editController.edit(itemID: item.id, text: text)
} }
} }
if case .image(let filename) = item.content { if case .image(let filename) = item.content, !multiple {
Button("Open") { Button("Open") {
NSWorkspace.shared.open(store.imageURL(for: filename)) NSWorkspace.shared.open(store.imageURL(for: filename))
} }
} }
if selection.count > 1, selection.contains(item.id) { if multiple {
Button("Merge Notes") { Button("Merge Notes") {
store.merge(ids: selection) store.merge(ids: ids)
clearSelection() clearSelection()
} }
} }
Menu("Move To") { Menu("Move To") {
ForEach(store.sections.filter { $0.id != store.sectionID(of: item.id) }) { section in ForEach(store.sections.filter { $0.id != store.sectionID(of: item.id) }) { section in
Button(section.name) { store.move(item.id, to: section.id) } Button(section.name) {
store.move(ids: ids, to: section.id)
if multiple { clearSelection() }
}
} }
} }
Divider() Divider()
Button("Delete", role: .destructive) { Button(multiple ? "Delete \(ids.count) Items" : "Delete", role: .destructive) {
selection.remove(item.id) store.delete(ids: ids)
store.delete(item.id) selection.subtract(ids)
lastSelectedID = nil
} }
} }
} }
@@ -664,12 +672,7 @@ struct PanelView: View {
private func handleClick(_ item: PromptItem) { private func handleClick(_ item: PromptItem) {
let modifiers = NSApp.currentEvent?.modifierFlags ?? [] let modifiers = NSApp.currentEvent?.modifierFlags ?? []
if modifiers.contains(.command) { if modifiers.contains(.command) {
if selection.contains(item.id) { toggleSelection(item.id)
selection.remove(item.id)
} else {
selection.insert(item.id)
}
lastSelectedID = item.id
return return
} }
if modifiers.contains(.shift), let anchor = lastSelectedID, anchor != item.id { if modifiers.contains(.shift), let anchor = lastSelectedID, anchor != item.id {
@@ -679,6 +682,15 @@ struct PanelView: View {
copy(item) copy(item)
} }
private func toggleSelection(_ id: UUID) {
if selection.contains(id) {
selection.remove(id)
} else {
selection.insert(id)
}
lastSelectedID = id
}
private func selectRange(from anchor: UUID, to target: UUID) { private func selectRange(from anchor: UUID, to target: UUID) {
let ids = displayedItems.map(\.item.id) let ids = displayedItems.map(\.item.id)
guard let start = ids.firstIndex(of: anchor), let end = ids.firstIndex(of: target) else { return } guard let start = ids.firstIndex(of: anchor), let end = ids.firstIndex(of: target) else { return }
+36 -5
View File
@@ -190,18 +190,49 @@ final class Store: ObservableObject {
} }
func delete(_ id: UUID) { func delete(_ id: UUID) {
delete(ids: [id])
}
/// Deletes every item in `ids` in one pass, cleaning up their image files.
func delete(ids: Set<UUID>) {
guard !ids.isEmpty else { return }
var removedAny = false
for sectionIndex in sections.indices { for sectionIndex in sections.indices {
if let itemIndex = sections[sectionIndex].items.firstIndex(where: { $0.id == id }) { for item in sections[sectionIndex].items where ids.contains(item.id) {
let item = sections[sectionIndex].items[itemIndex] removedAny = true
if case .image(let filename) = item.content { if case .image(let filename) = item.content {
try? FileManager.default.removeItem(at: imagesDirectory.appendingPathComponent(filename)) try? FileManager.default.removeItem(at: imagesDirectory.appendingPathComponent(filename))
imageCache.removeObject(forKey: filename as NSString) imageCache.removeObject(forKey: filename as NSString)
} }
sections[sectionIndex].items.remove(at: itemIndex) }
save() sections[sectionIndex].items.removeAll { ids.contains($0.id) }
return }
if removedAny { save() }
}
func setDone(_ done: Bool, ids: Set<UUID>) {
guard !ids.isEmpty else { return }
for sectionIndex in sections.indices {
for itemIndex in sections[sectionIndex].items.indices
where ids.contains(sections[sectionIndex].items[itemIndex].id) {
sections[sectionIndex].items[itemIndex].done = done
} }
} }
save()
}
/// Moves every item in `ids` into `targetSectionID`, keeping their order.
func move(ids: Set<UUID>, to targetSectionID: UUID) {
guard !ids.isEmpty,
let targetIndex = sections.firstIndex(where: { $0.id == targetSectionID }) else { return }
var moved: [PromptItem] = []
for sectionIndex in sections.indices where sectionIndex != targetIndex {
moved.append(contentsOf: sections[sectionIndex].items.filter { ids.contains($0.id) })
sections[sectionIndex].items.removeAll { ids.contains($0.id) }
}
guard !moved.isEmpty else { return }
sections[targetIndex].items.append(contentsOf: moved)
save()
} }
func move(_ id: UUID, to targetSectionID: UUID) { func move(_ id: UUID, to targetSectionID: UUID) {