Breaking change: don't initially show preview (#348)

kelleyma49 · web-flow · commit fe06a3d4a4b5 · 2025-09-20T16:01:05.000-07:00
- Makes it more consistent with file and history commands - Intended to resolve #264
diff --git a/PSFzf.TabExpansion.ps1 b/PSFzf.TabExpansion.ps1
@@ -282,13 +282,13 @@ function script:Invoke-FzfTabCompletionInner() {
         $arguments = @{
             Layout        = 'reverse'
             Expect        = "$expectTriggers"
-            PreviewWindow = 'down:30%'
+            PreviewWindow = 'hidden'
         }
         if ($isTabTrigger) {
-            $arguments["Bind"] = @('ctrl-/:change-preview-window(down,right:50%,border-top|hidden|)')
+            $arguments["Bind"] = @('ctrl-/:change-preview-window:down|right|right:hidden')
         }
         else {
-            $arguments["Bind"] = @('tab:down', 'btab:up', 'ctrl-/:change-preview-window(down,right:50%,border-top|hidden|)')
+            $arguments["Bind"] = @('tab:down', 'btab:up', 'ctrl-/:change-preview-window:down|right|right:hidden')
         }
 
         # need to handle parameters differently so PowerShell doesn't parse completion item as a script parameter:

Original file line number	Diff line number	Diff line change
`@@ -282,13 +282,13 @@ function script:Invoke-FzfTabCompletionInner() {`
`282`	`282`	`$arguments = @{`
`283`	`283`	`Layout = 'reverse'`
`284`	`284`	`Expect = "$expectTriggers"`
`285`		`- PreviewWindow = 'down:30%'`
	`285`	`+ PreviewWindow = 'hidden'`
`286`	`286`	`}`
`287`	`287`	`if ($isTabTrigger) {`
`288`		`- $arguments["Bind"] = @('ctrl-/:change-preview-window(down,right:50%,border-top\|hidden\|)')`
	`288`	`+ $arguments["Bind"] = @('ctrl-/:change-preview-window:down\|right\|right:hidden')`
`289`	`289`	`}`
`290`	`290`	`else {`
`291`		`- $arguments["Bind"] = @('tab:down', 'btab:up', 'ctrl-/:change-preview-window(down,right:50%,border-top\|hidden\|)')`
	`291`	`+ $arguments["Bind"] = @('tab:down', 'btab:up', 'ctrl-/:change-preview-window:down\|right\|right:hidden')`
`292`	`292`	`}`
`293`	`293`
`294`	`294`	`# need to handle parameters differently so PowerShell doesn't parse completion item as a script parameter:`