Phase 3 finishes the pivot at the surface. One card face serves two containers: CardFaceView extracted with a role — board or trash — so stripe, tint, chip, selection stroke, cut dim, marquee registration, and drag are shared by construction, the trash side differing only in its absences: no Open, no rename, no Style, no file-hover highlight, and a Delete that goes through the confirmation host. The column rewrote around the lanes' own single-column masonry so drag reflow reads as positional slides; chrome stays the hatched header, symbol, and count — 11 gives Empty Trash to the File menu alone. Two real grammar bugs die here: plain Backspace on a trash selection purged without the confirmation the menu raises, and the context menu's Delete resolved against the standing selection, so right-clicking a trash card under a board selection silently did nothing — it now stages the clicked set explicitly. Open, Rename, Style, and Empty Trash validation became testable store seams; the column is one named accessibility container of ordinary card elements. The tombstone era is swept: deleteItem, restoreItem, stripTombstonedChildren — dead since lane copies stopped nesting trash — the restore verb, the unreachable put-back banner row, and every quasi-lane doc comment. Claude-Session: https://claude.ai/code/session_01SR4XGjmBE16ZUYWpfFHXwY
110 lines
6.4 KiB
Swift
110 lines
6.4 KiB
Swift
/// 04-interactions.md's **⌘N target rule** (settled), as a pure function of the three things it
|
|
/// reads — the selection, the last-active lane, and the snapshot (`NewCardTargetTests`).
|
|
///
|
|
/// The rule verbatim, and each clause's branch below:
|
|
///
|
|
/// > with a card selected, the new card is created in that card's lane, immediately after it
|
|
/// > (paste-anchor consistency); with a lane selected, appended at its bottom (Return consistency);
|
|
/// > a multi-selection anchors at its last member in flatten order (lane `order`, then card
|
|
/// > `order`, the multi-drag order; the same anchor serves paste): creation follows the last
|
|
/// > selected card, or appends to the last selected lane; … with nothing selected — or a
|
|
/// > **trash** selection, which never anchors creation — the **last-active lane** — the lane
|
|
/// > that most recently held selection or a creation in this window session — falling back to the
|
|
/// > first lane. … **Zero-lane board**: card creation … disable[s] via menu validation until a
|
|
/// > lane exists.
|
|
///
|
|
/// **A pure function rather than a method on the store** for the reason every rule in this codebase
|
|
/// that can be one is: the branches are lines of test rather than UI states to drive, and the menu
|
|
/// item's `disabled` and its action then read the *same* answer instead of two hand-kept-in-sync
|
|
/// conditions.
|
|
///
|
|
/// ### What it deliberately does not decide
|
|
///
|
|
/// - **The lane header's new-card button overrides this rule entirely** (11-command-nexus.md ▸
|
|
/// Pointer grammar, settled): "the click names its target lane, selection notwithstanding". That
|
|
/// call site passes its own lane and never comes here.
|
|
/// - **Return on a selected lane** is the same target as this rule's lane branch, but it is reached
|
|
/// by grammar rather than by the menu; it also passes its lane directly.
|
|
enum NewCardTarget {
|
|
|
|
/// Where a new card goes: which lane, and which card it lands immediately after (`nil` = the
|
|
/// lane's bottom). Exactly `NewCardPlaceholder`'s two anchoring fields, because that is what
|
|
/// this resolves *into*.
|
|
struct Resolution: Equatable {
|
|
let laneID: ItemID
|
|
let anchorCardID: ItemID?
|
|
}
|
|
|
|
/// The target, or `nil` when there is none — **the zero-lane board**, where "New Card,
|
|
/// Return-creation, and Paste with a card payload disable via menu validation until a lane
|
|
/// exists". `nil` is therefore the menu item's `disabled` condition as well as its refusal, so
|
|
/// the two can never disagree.
|
|
///
|
|
/// - Parameters:
|
|
/// - selection: the board's current selection, container included. A `.trash` selection
|
|
/// "never anchors creation" and is treated exactly as an empty one — the settled precedent
|
|
/// 04 ▸ Clipboard cites for paste, applied here to its source rule ("a trashed card's live
|
|
/// disk-lane never leaks in as 'the selected card's lane'").
|
|
/// - lastActiveLaneID: `TransientBoardState.lastActiveLaneID`, already cleared by the reload
|
|
/// rule if its lane vanished — but re-checked here anyway, because a caller need not have
|
|
/// reloaded since the lane went.
|
|
static func resolve(
|
|
selection: ItemReferenceSet,
|
|
lastActiveLaneID: ItemID?,
|
|
snapshot: BoardModel
|
|
) -> Resolution? {
|
|
let lanes = snapshot.lanes
|
|
guard !lanes.isEmpty else { return nil }
|
|
|
|
if let anchor = flattenAnchor(selection: selection, snapshot: snapshot) {
|
|
return anchor
|
|
}
|
|
// Nothing selected, a trash selection, or a stale one — the ids name nothing the board
|
|
// renders, a selection the next reload will drop. Falls through rather than refusing: the
|
|
// user pressed ⌘N and the board has lanes. The target is then the lane that most recently
|
|
// held selection or a creation, and the first lane when there is no such lane (or it has
|
|
// since gone).
|
|
if let lastActiveLaneID, let lane = lanes.first(where: { $0.id == lastActiveLaneID }) {
|
|
return Resolution(laneID: lane.id, anchorCardID: nil)
|
|
}
|
|
return lanes.first.map { Resolution(laneID: $0.id, anchorCardID: nil) }
|
|
}
|
|
|
|
/// **The shared anchor, on its own** — "a multi-selection anchors at its last member in flatten
|
|
/// order (lane `order`, then card `order`, the multi-drag order; the same anchor serves paste)".
|
|
///
|
|
/// Extracted rather than left inside `resolve` because paste needs *exactly this clause* and not
|
|
/// the two that surround it. Card paste is `resolve` verbatim (the last-active-lane fallback and
|
|
/// all), but **lane paste has a different fallback** — "nothing selected = the board's right end",
|
|
/// never the last-active lane — so it takes the anchor and stops. Two derivations of "the last
|
|
/// member in flatten order" would be two chances for creation and paste to disagree about the one
|
|
/// rule 04 says they share.
|
|
///
|
|
/// `nil` covers the three cases that anchor nothing, which the callers then answer their own way:
|
|
/// an empty selection, a **trash** one ("a trash selection never anchors paste",
|
|
/// settled — and "a trashed card's live disk-lane never leaks in as 'the selected card's lane'",
|
|
/// which falls out of never looking at the trashed side at all), and a stale one whose ids name
|
|
/// nothing the board renders.
|
|
static func flattenAnchor(selection: ItemReferenceSet, snapshot: BoardModel) -> Resolution? {
|
|
guard selection.container == .board, !selection.ids.isEmpty else { return nil }
|
|
|
|
// The snapshot's lanes and cards are already in display order, so the flatten order is one
|
|
// walk, and the *last* hit is the anchor. Selection is homogeneous (cards XOR lanes), so only
|
|
// one of the two branches ever fires within a walk; a sole selection is simply the degenerate
|
|
// one-member case of the same rule.
|
|
var anchor: Resolution?
|
|
for lane in snapshot.lanes {
|
|
// A selected lane: creation appends at its bottom, Return consistency; paste lands after
|
|
// the lane itself.
|
|
if selection.ids.contains(lane.id) {
|
|
anchor = Resolution(laneID: lane.id, anchorCardID: nil)
|
|
}
|
|
// A selected card: its lane, immediately after it — paste-anchor consistency.
|
|
for card in lane.cards where selection.ids.contains(card.id) {
|
|
anchor = Resolution(laneID: lane.id, anchorCardID: card.id)
|
|
}
|
|
}
|
|
return anchor
|
|
}
|
|
}
|