feat: zoom-center-on-cursor, deep-clone helper, UI polish

aksOps · claude · aksOps · commit 95f96c4c78d5 · 2026-04-17T14:05:35.000Z
Core helpers + their tests (all Linux-verifiable)
  * Get-ZoomCenteredOffset — pure math for Ctrl+MouseWheel zoom
    that preserves the content point under the cursor across a
    scale change, clamped to [0, Content-Viewport]. Previously the
    wheel handler only changed scale and left the scroll offset
    alone, so the README claim "Zoom centered on cursor" wasn't
    actually implemented. Wire it into the handler.
  * Copy-AnnotationList — pull the three identical deep-clone
    loops out of Snapshot-State / Do-Undo / Do-Redo into a single
    pure helper; tests cover null/empty input, single-annotation
    mutation isolation, mixed-type round-trip.
  * Headless tests: 72 -&gt; 84 (+6 zoom math + 6 annotation clone).

XAML polish (parse-verified, WPF eye-test still recommended)
  * Bump TextSecondaryBrush 0xAA -&gt; 0xDD and About window
    #88FFFFFF / #BBBBBB -&gt; #BBFFFFFF / #DDDDDD for WCAG AA.
  * Tiny center dot in the magnifier loupe for pixel-precise aim.
  * &lt;ResizeGrip&gt; at preview bottom-right — makes the edge-drag
    affordance discoverable on chromeless windows.
  * MinWidth/MinHeight on the About window so narrow DPI setups
    don't clip the caption buttons.
  * Copy button now flashes "Copied!" for 1.2s after clipboard set.
  * Active color swatch: 30x30 with a 3px accent ring (was 26x26
    with a 2px white ring) — the selection is now obvious.

CI hygiene
  * FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true at workflow level
    clears the actions/checkout@v4 Node 20 deprecation notice
    ahead of the 2026-09 removal.

All 4 embedded XAML blocks still parse cleanly via [xml] and
AST parser.

Co-Authored-By: Claude Opus 4.7 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -7,6 +7,11 @@ on:
     branches: [main]
   workflow_dispatch:
 
+# Opt in to Node 24 runtime for actions/* (Node 20 is deprecated;
+# forced default in June 2026, removed in September 2026).
+env:
+  FORCE_JAVASCRIPT_ACTIONS_TO_NODE24: "true"
+
 jobs:
   headless:
     name: Pure-logic tests (pwsh 7.5+)
diff --git a/SnipIT.ps1 b/SnipIT.ps1
@@ -183,6 +183,54 @@ function Get-ClampedAnnotationRect {
     [pscustomobject]@{ X = $nx; Y = $ny; Width = $nw; Height = $nh }
 }
 
+function Get-ZoomCenteredOffset {
+    # Compute the scroll offset that keeps the content point currently under
+    # ($CursorX, $CursorY) anchored after a scale change (Ctrl+MouseWheel zoom).
+    # Content coords = viewport-offset + viewport-position; if the same image
+    # pixel should land on the same viewport pixel after scaling, the offset
+    # must shift by (OldOffset + Cursor) * NewScale/OldScale - Cursor.
+    # Result is clamped to [0, Content - Viewport].
+    param(
+        [Parameter(Mandatory)] [double]$CursorX,
+        [Parameter(Mandatory)] [double]$CursorY,
+        [Parameter(Mandatory)] [double]$OldScrollX,
+        [Parameter(Mandatory)] [double]$OldScrollY,
+        [Parameter(Mandatory)] [double]$OldScale,
+        [Parameter(Mandatory)] [double]$NewScale,
+        [Parameter(Mandatory)] [double]$ContentWidth,
+        [Parameter(Mandatory)] [double]$ContentHeight,
+        [Parameter(Mandatory)] [double]$ViewportWidth,
+        [Parameter(Mandatory)] [double]$ViewportHeight
+    )
+    if ($OldScale -le 0) { $OldScale = 1.0 }
+    $ratio = $NewScale / $OldScale
+    $newX = ($OldScrollX + $CursorX) * $ratio - $CursorX
+    $newY = ($OldScrollY + $CursorY) * $ratio - $CursorY
+    $maxX = [math]::Max(0.0, $ContentWidth  - $ViewportWidth)
+    $maxY = [math]::Max(0.0, $ContentHeight - $ViewportHeight)
+    $newX = [math]::Max(0.0, [math]::Min($maxX, $newX))
+    $newY = [math]::Max(0.0, [math]::Min($maxY, $newY))
+    [pscustomobject]@{ X = $newX; Y = $newY }
+}
+
+function Copy-AnnotationList {
+    # Deep-clone an enumerable of annotation pscustomobjects into a new
+    # ArrayList. ArrayList stores references, so Snapshot-State / Do-Undo /
+    # Do-Redo need fresh copies to avoid letting undo entries mutate each
+    # other when the live Annotations list is edited.
+    param([AllowNull()][AllowEmptyCollection()] $Annotations)
+    $copy = New-Object System.Collections.ArrayList
+    if ($null -eq $Annotations) { return ,$copy }
+    foreach ($a in $Annotations) {
+        [void]$copy.Add([pscustomobject]@{
+            Type=$a.Type; Color=$a.Color
+            X=$a.X; Y=$a.Y; W=$a.W; H=$a.H
+            Text=$a.Text; FontSize=$a.FontSize
+        })
+    }
+    return ,$copy
+}
+
 function Get-TrimmedRecent {
     # Keep only the top N items (for capping unbounded undo/redo stacks).
     # $Items is expected in most-recent-first order, matching [Stack].ToArray().
@@ -545,7 +593,7 @@ function Show-AboutWindow {
     [xml]$xaml = @"
 <Window xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
         xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
-        Title="About SnipIT" Width="420" Height="300"
+        Title="About SnipIT" Width="420" Height="300" MinWidth="360" MinHeight="280"
         WindowStartupLocation="CenterScreen"
         WindowStyle="None" AllowsTransparency="True" ResizeMode="NoResize"
         Background="#FF1B1B1B">
@@ -565,7 +613,7 @@ function Show-AboutWindow {
     </Border>
     <StackPanel Grid.Row="1" Margin="24,18,24,0">
       <TextBlock Text="Professional snipping tool" FontSize="13" Foreground="#CCFFFFFF" Margin="0,0,0,14"/>
-      <TextBlock Text="PowerShell 7.5+ on .NET 9" FontSize="11" Foreground="#88FFFFFF" Margin="0,0,0,16"/>
+      <TextBlock Text="PowerShell 7.5+ on .NET 9" FontSize="11" Foreground="#BBFFFFFF" Margin="0,0,0,16"/>
       <TextBlock Text="Hotkeys" FontSize="12" FontWeight="SemiBold" Foreground="White" Margin="0,0,0,6"/>
       <Grid>
         <Grid.ColumnDefinitions>
@@ -576,11 +624,11 @@ function Show-AboutWindow {
           <RowDefinition/><RowDefinition/><RowDefinition/>
         </Grid.RowDefinitions>
         <TextBlock Grid.Row="0" Grid.Column="0" Text="Ctrl + Shift + S" FontFamily="Consolas" Foreground="#DDDDDD"/>
-        <TextBlock Grid.Row="0" Grid.Column="1" Text="Smart capture"   Foreground="#BBBBBB"/>
+        <TextBlock Grid.Row="0" Grid.Column="1" Text="Smart capture"   Foreground="#DDDDDD"/>
         <TextBlock Grid.Row="1" Grid.Column="0" Text="Ctrl + Shift + F" FontFamily="Consolas" Foreground="#DDDDDD"/>
-        <TextBlock Grid.Row="1" Grid.Column="1" Text="Full screen"     Foreground="#BBBBBB"/>
+        <TextBlock Grid.Row="1" Grid.Column="1" Text="Full screen"     Foreground="#DDDDDD"/>
         <TextBlock Grid.Row="2" Grid.Column="0" Text="Ctrl + Shift + W" FontFamily="Consolas" Foreground="#DDDDDD"/>
-        <TextBlock Grid.Row="2" Grid.Column="1" Text="Active window"   Foreground="#BBBBBB"/>
+        <TextBlock Grid.Row="2" Grid.Column="1" Text="Active window"   Foreground="#DDDDDD"/>
       </Grid>
     </StackPanel>
     <Border Grid.Row="2" Padding="20,12" Background="#22000000">
@@ -645,6 +693,9 @@ function Show-SmartOverlay {
                      RenderOptions.BitmapScalingMode="NearestNeighbor"/>
               <Line X1="72" Y1="0" X2="72" Y2="144" Stroke="#990078D4" StrokeThickness="1"/>
               <Line X1="0"  Y1="72" X2="144" Y2="72" Stroke="#990078D4" StrokeThickness="1"/>
+              <!-- Tiny center dot — marks the exact pixel under the cursor -->
+              <Rectangle Width="4" Height="4" Fill="#FFFF4081"
+                         HorizontalAlignment="Center" VerticalAlignment="Center"/>
             </Grid>
           </Border>
           <TextBlock x:Name="LoupeText" Foreground="White" FontFamily="Consolas"
@@ -842,7 +893,7 @@ function Show-PreviewWindow {
     <SolidColorBrush x:Key="AccentBrush"        Color="#FF0078D4"/>
     <SolidColorBrush x:Key="AccentHoverBrush"   Color="#330078D4"/>
     <SolidColorBrush x:Key="PanelOverlayBrush"  Color="#22000000"/>
-    <SolidColorBrush x:Key="TextSecondaryBrush" Color="#AAFFFFFF"/>
+    <SolidColorBrush x:Key="TextSecondaryBrush" Color="#DDFFFFFF"/>
     <SolidColorBrush x:Key="BorderLightBrush"   Color="#33FFFFFF"/>
 
     <!-- Active-tool visual state for the 4 annotation tools + Pin.
@@ -985,6 +1036,11 @@ function Show-PreviewWindow {
         </Button>
       </StackPanel>
     </Border>
+
+    <!-- Resize grip affordance. Chromeless windows still accept edge-drag,
+         but this makes the corner discoverable. -->
+    <ResizeGrip Grid.Row="3" HorizontalAlignment="Right" VerticalAlignment="Bottom"
+                Width="14" Height="14" Opacity="0.55" IsTabStop="False"/>
   </Grid>
 </Window>
 "@
@@ -1151,63 +1207,30 @@ function Show-PreviewWindow {
     }
 
     function script:Snapshot-State {
-        # Deep-copy current annotations into undo stack, clear redo
-        $copy = New-Object System.Collections.ArrayList
-        foreach ($a in $state.Annotations) {
-            [void]$copy.Add([pscustomobject]@{
-                Type=$a.Type; Color=$a.Color
-                X=$a.X; Y=$a.Y; W=$a.W; H=$a.H
-                Text=$a.Text; FontSize=$a.FontSize
-            })
-        }
-        $state.UndoStack.Push($copy)
+        $state.UndoStack.Push( (Copy-AnnotationList $state.Annotations) )
         $state.RedoStack.Clear()
         Trim-SnipStack $state.UndoStack
     }
 
     function script:Restore-State {
         param($snapshot)
         $state.Annotations.Clear()
-        foreach ($a in $snapshot) {
-            [void]$state.Annotations.Add([pscustomobject]@{
-                Type=$a.Type; Color=$a.Color
-                X=$a.X; Y=$a.Y; W=$a.W; H=$a.H
-                Text=$a.Text; FontSize=$a.FontSize
-            })
-        }
+        foreach ($a in (Copy-AnnotationList $snapshot)) { [void]$state.Annotations.Add($a) }
         Render-Annotations
     }
 
     function script:Do-Undo {
         if ($state.UndoStack.Count -eq 0) { return }
-        $current = New-Object System.Collections.ArrayList
-        foreach ($a in $state.Annotations) {
-            [void]$current.Add([pscustomobject]@{
-                Type=$a.Type; Color=$a.Color
-                X=$a.X; Y=$a.Y; W=$a.W; H=$a.H
-                Text=$a.Text; FontSize=$a.FontSize
-            })
-        }
-        $state.RedoStack.Push($current)
+        $state.RedoStack.Push( (Copy-AnnotationList $state.Annotations) )
         Trim-SnipStack $state.RedoStack
-        $prev = $state.UndoStack.Pop()
-        Restore-State $prev
+        Restore-State $state.UndoStack.Pop()
     }
 
     function script:Do-Redo {
         if ($state.RedoStack.Count -eq 0) { return }
-        $current = New-Object System.Collections.ArrayList
-        foreach ($a in $state.Annotations) {
-            [void]$current.Add([pscustomobject]@{
-                Type=$a.Type; Color=$a.Color
-                X=$a.X; Y=$a.Y; W=$a.W; H=$a.H
-                Text=$a.Text; FontSize=$a.FontSize
-            })
-        }
-        $state.UndoStack.Push($current)
+        $state.UndoStack.Push( (Copy-AnnotationList $state.Annotations) )
         Trim-SnipStack $state.UndoStack
-        $next = $state.RedoStack.Pop()
-        Restore-State $next
+        Restore-State $state.RedoStack.Pop()
     }
 
     # Named color picker. Tests and the real swatch click handler both call
@@ -1237,18 +1260,24 @@ function Show-PreviewWindow {
         $colorBar.Children.Clear()
         foreach ($name in $palette.Keys) {
             $rgb = $palette[$name]
+            $isActive = ($state.ActiveColor -eq $name)
             $sw = New-Object System.Windows.Controls.Border
-            $sw.Width = 26; $sw.Height = 26
+            # Slightly larger + accent-coloured ring when active — makes the
+            # selected swatch unambiguous at a glance.
+            if ($isActive) { $sw.Width = 30; $sw.Height = 30 }
+            else           { $sw.Width = 26; $sw.Height = 26 }
             $sw.Margin = New-Object System.Windows.Thickness 3, 0, 3, 0
             $sw.CornerRadius = New-Object System.Windows.CornerRadius 0
             $sw.Background = New-Object System.Windows.Media.SolidColorBrush(
                 (To-WpfColor 255 $rgb.R $rgb.G $rgb.B))
-            $sw.BorderBrush = New-Object System.Windows.Media.SolidColorBrush(
-                ([System.Windows.Media.Colors]::White))
-            $sw.BorderThickness = if ($state.ActiveColor -eq $name) {
-                New-Object System.Windows.Thickness 2
+            if ($isActive) {
+                $sw.BorderBrush = New-Object System.Windows.Media.SolidColorBrush(
+                    ([System.Windows.Media.Color]::FromArgb(255, 0x00, 0x78, 0xD4)))
+                $sw.BorderThickness = New-Object System.Windows.Thickness 3
             } else {
-                New-Object System.Windows.Thickness 0
+                $sw.BorderBrush = New-Object System.Windows.Media.SolidColorBrush(
+                    ([System.Windows.Media.Colors]::White))
+                $sw.BorderThickness = New-Object System.Windows.Thickness 0
             }
             $sw.Cursor = [System.Windows.Input.Cursors]::Hand
             # Non-focusable so clicking a swatch while a text box is open
@@ -1719,8 +1748,23 @@ function Show-PreviewWindow {
 
     $win.Add_PreviewMouseWheel({
         if (([System.Windows.Input.Keyboard]::Modifiers -band [System.Windows.Input.ModifierKeys]::Control) -ne 0) {
-            $factor = if ($_.Delta -gt 0) { 1.25 } else { 1 / 1.25 }
+            $factor    = if ($_.Delta -gt 0) { 1.25 } else { 1 / 1.25 }
+            $oldScale  = $layoutScale.ScaleX
+            $cursor    = $_.GetPosition($scroller)
+            $oldSx     = $scroller.HorizontalOffset
+            $oldSy     = $scroller.VerticalOffset
             & $zoomBy $factor
+            $newScale  = $layoutScale.ScaleX
+            $offset    = Get-ZoomCenteredOffset `
+                -CursorX $cursor.X -CursorY $cursor.Y `
+                -OldScrollX $oldSx -OldScrollY $oldSy `
+                -OldScale $oldScale -NewScale $newScale `
+                -ContentWidth  ($Bitmap.Width  * $newScale) `
+                -ContentHeight ($Bitmap.Height * $newScale) `
+                -ViewportWidth  $scroller.ViewportWidth `
+                -ViewportHeight $scroller.ViewportHeight
+            $scroller.ScrollToHorizontalOffset($offset.X)
+            $scroller.ScrollToVerticalOffset(  $offset.Y)
             $_.Handled = $true
         }
     }.GetNewClosure())
@@ -1793,12 +1837,20 @@ function Show-PreviewWindow {
         return $flat
     }
 
-    $win.FindName('CopyBtn').Add_Click({
+    $copyBtn = $win.FindName('CopyBtn')
+    $copyBtn.Add_Click({
         $flat = Get-FlattenedBitmap
         $clipSrc = Convert-BitmapToBitmapSource $flat
         [System.Windows.Clipboard]::SetImage($clipSrc)
         if ($flat -ne $Bitmap) { $flat.Dispose() }
-    })
+        # Brief confirmation — second StackPanel child is the "Copy" label.
+        $lbl = $copyBtn.Content.Children[1]
+        $lbl.Text = 'Copied!'
+        $timer = New-Object System.Windows.Threading.DispatcherTimer
+        $timer.Interval = [timespan]::FromMilliseconds(1200)
+        $timer.Add_Tick({ $lbl.Text = 'Copy'; $timer.Stop() }.GetNewClosure())
+        $timer.Start()
+    }.GetNewClosure())
     $win.FindName('SaveBtn').Add_Click({
         $flat = Get-FlattenedBitmap
         Save-CaptureToFile -Bitmap $flat | Out-Null
diff --git a/Test-SnipIT.ps1 b/Test-SnipIT.ps1
@@ -209,6 +209,118 @@ It 'is case-insensitive for extension recognition' {
     ShouldBe (Resolve-SaveImagePath -Path '/tmp/a.PNG' -FilterFormat 'Jpeg') '/tmp/a.PNG'
 }
 
+Describe 'Get-ZoomCenteredOffset'
+It 'keeps the same content point under the cursor at center of viewport' {
+    # Cursor at (400, 300) in viewport. Old offset 0, old scale 1, new scale 2.
+    # Content point under cursor before zoom: (400, 300). After 2x it's at (800, 600).
+    # New offset must shift so content (800, 600) maps back to viewport (400, 300).
+    $o = Get-ZoomCenteredOffset -CursorX 400 -CursorY 300 `
+        -OldScrollX 0 -OldScrollY 0 -OldScale 1 -NewScale 2 `
+        -ContentWidth 1920 -ContentHeight 1080 `
+        -ViewportWidth 800 -ViewportHeight 600
+    ShouldBe $o.X 400; ShouldBe $o.Y 300
+}
+It 'zooming in near the right edge clamps to the content boundary' {
+    # Cursor near right edge; after 2x zoom, the computed offset would exceed content-viewport.
+    $o = Get-ZoomCenteredOffset -CursorX 790 -CursorY 590 `
+        -OldScrollX 1000 -OldScrollY 400 -OldScale 1 -NewScale 2 `
+        -ContentWidth 2000 -ContentHeight 800 `
+        -ViewportWidth 800 -ViewportHeight 600
+    ShouldBe $o.X 1200   # max = ContentW - ViewportW = 2000 - 800
+    ShouldBe $o.Y 200    # max = ContentH - ViewportH = 800 - 600
+}
+It 'zooming out past the content fit clamps to zero' {
+    # Zoom from 2x down to 0.5x in a small image — no room to scroll.
+    $o = Get-ZoomCenteredOffset -CursorX 100 -CursorY 100 `
+        -OldScrollX 50 -OldScrollY 50 -OldScale 2 -NewScale 0.5 `
+        -ContentWidth 200 -ContentHeight 200 `
+        -ViewportWidth 400 -ViewportHeight 400
+    ShouldBe $o.X 0; ShouldBe $o.Y 0
+}
+It 'handles a zero OldScale gracefully (treats it as 1)' {
+    # Degenerate input; earlier zoom was a no-op. Should not divide by zero.
+    $o = Get-ZoomCenteredOffset -CursorX 50 -CursorY 50 `
+        -OldScrollX 0 -OldScrollY 0 -OldScale 0 -NewScale 2 `
+        -ContentWidth 1000 -ContentHeight 1000 `
+        -ViewportWidth 400 -ViewportHeight 400
+    ShouldBe $o.X 50; ShouldBe $o.Y 50
+}
+It 'a no-op scale change leaves the offset untouched' {
+    $o = Get-ZoomCenteredOffset -CursorX 123 -CursorY 456 `
+        -OldScrollX 77 -OldScrollY 88 -OldScale 1.5 -NewScale 1.5 `
+        -ContentWidth 2000 -ContentHeight 1500 `
+        -ViewportWidth 800 -ViewportHeight 600
+    ShouldBe $o.X 77; ShouldBe $o.Y 88
+}
+It 'matrix: 0.5x / 1x / 2x / 5x at viewport center produces sensible offsets' {
+    $row = 300; $col = 400
+    foreach ($s in 0.5, 1.0, 2.0, 5.0) {
+        $o = Get-ZoomCenteredOffset -CursorX $col -CursorY $row `
+            -OldScrollX 0 -OldScrollY 0 -OldScale 1 -NewScale $s `
+            -ContentWidth (4000) -ContentHeight (3000) `
+            -ViewportWidth 800 -ViewportHeight 600
+        # Expected: cursor*(s-1); clamped to [0, 4000-800] and [0, 3000-600]
+        $expectedX = [math]::Max(0.0, [math]::Min(3200.0, $col * ($s - 1)))
+        $expectedY = [math]::Max(0.0, [math]::Min(2400.0, $row * ($s - 1)))
+        ShouldBe $o.X $expectedX
+        ShouldBe $o.Y $expectedY
+    }
+}
+
+Describe 'Copy-AnnotationList'
+It 'returns an empty ArrayList for null input' {
+    $r = Copy-AnnotationList $null
+    ShouldBe $r.Count 0
+}
+It 'returns an empty ArrayList for an empty input' {
+    $r = Copy-AnnotationList @()
+    ShouldBe $r.Count 0
+}
+It 'deep-copies a single highlight annotation' {
+    $src = @([pscustomobject]@{ Type='highlight'; Color='yellow'; X=10; Y=20; W=100; H=50; Text=$null; FontSize=0 })
+    $r = Copy-AnnotationList $src
+    ShouldBe $r.Count 1
+    ShouldBe $r[0].Type 'highlight'
+    ShouldBe $r[0].X 10
+}
+It 'mutations on the copy do not affect the original' {
+    $orig = @([pscustomobject]@{ Type='rect'; Color='red'; X=5; Y=5; W=50; H=50; Text=$null; FontSize=0 })
+    $copy = Copy-AnnotationList $orig
+    $copy[0].X = 999
+    ShouldBe $orig[0].X 5    # original untouched
+    ShouldBe $copy[0].X 999
+}
+It 'preserves mixed annotation types (highlight + rect + arrow + text)' {
+    $src = @(
+        [pscustomobject]@{ Type='highlight'; Color='yellow'; X=0;  Y=0;  W=10; H=10; Text=$null;   FontSize=0 }
+        [pscustomobject]@{ Type='rect';      Color='blue';   X=10; Y=10; W=20; H=20; Text=$null;   FontSize=0 }
+        [pscustomobject]@{ Type='arrow';     Color='red';    X=20; Y=20; W=30; H=30; Text=$null;   FontSize=0 }
+        [pscustomobject]@{ Type='text';      Color='green';  X=30; Y=30; W=40; H=40; Text='hello'; FontSize=24 }
+    )
+    $r = Copy-AnnotationList $src
+    ShouldBe $r.Count 4
+    ShouldBe $r[0].Type 'highlight'
+    ShouldBe $r[1].Type 'rect'
+    ShouldBe $r[2].Type 'arrow'
+    ShouldBe $r[3].Type 'text'
+    ShouldBe $r[3].Text 'hello'
+    ShouldBe $r[3].FontSize 24
+}
+It 'undo-then-redo round trip: a sequence of copies yields identical content' {
+    $original = @(
+        [pscustomobject]@{ Type='highlight'; Color='yellow'; X=1; Y=2; W=3; H=4; Text=$null; FontSize=0 }
+        [pscustomobject]@{ Type='text';      Color='red';    X=5; Y=6; W=7; H=8; Text='hi';  FontSize=16 }
+    )
+    $snap1 = Copy-AnnotationList $original      # undo entry
+    $snap2 = Copy-AnnotationList $snap1         # redo entry after "undo"
+    $snap3 = Copy-AnnotationList $snap2         # restored after "redo"
+    ShouldBe $snap3.Count 2
+    ShouldBe $snap3[0].X 1
+    ShouldBe $snap3[1].Text 'hi'
+    # And the original is untouched throughout
+    ShouldBe $original[0].X 1
+}
+
 Describe 'Get-ClampedAnnotationRect'
 It 'passes through a rect that is fully inside' {
     $r = Get-ClampedAnnotationRect -X 10 -Y 20 -Width 100 -Height 50 `
