Skip to content

Commit 3d2c10a

Browse files
Address review feedback: clear the scrub pill on every session-ending path
endSession(resync:) is the choke point for revertEditing, abortEditingForExternalPick, and finishEditing's fallback paths, but only finishDragOrScrollSession cleared rowScrubPreview. Escape-cancelling a scrub left the pill frozen on screen.
1 parent d9bb755 commit 3d2c10a

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

Pika/Views/EditableColorValue.swift

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -521,6 +521,7 @@ struct EditableColorValue: View {
521521
isInvalid = false
522522
preEditColor = nil
523523
sessionOwner = nil
524+
rowScrubPreview = nil
524525
// Only clear the own-write marker when we're about to resync anyway. A `resync: false`
525526
// commit (see `finishEditing`'s success path) deliberately keeps `values` as typed rather
526527
// than the freshly (and possibly lossily) decomposed colour — clearing this unconditionally

0 commit comments

Comments
 (0)