Skip to content

Follow-Up to #47: Configure native apps for inline previews - #51

Draft
alvst wants to merge 74 commits into
momenbasel:mainfrom
alvst:codex/preview-polish-followup
Draft

Follow-Up to #47: Configure native apps for inline previews#51
alvst wants to merge 74 commits into
momenbasel:mainfrom
alvst:codex/preview-polish-followup

Conversation

@alvst

@alvst alvst commented Jul 29, 2026

Copy link
Copy Markdown

Summary

Configures the native “Open in …” action for inline previews.

  • Adds a compact split-button: the primary action opens in the configured default app, while the menu offers a short set of compatible suggested apps and a one-off chooser.
  • Lets people set separate defaults for text/rich text, images, and links in Settings. Defaults remain TextEdit, Preview, and Safari.
  • Opens exported temporary copies for text and images, so external edits never mutate Pesty-managed history files.
  • Restricts link suggestions to applications that explicitly declare HTTP/HTTPS handling, avoiding generic utilities such as ColorSync in the browser list.
  • Keys the native menu to the active clip so a previous card’s app list cannot leak into the next preview.

Follow-up

Follow-up to #47, Open preview content in native apps. #47 itself follows #45 and #46, which provide the detached inline-preview surface.

Dependency

This draft is based on #49’s Paste Bar interaction work. Merge #47 and #49 first; its own change is the final preview-app configuration commit.

Validation

  • swift build
  • git diff --check

Screenshot

Debug Demo (Pesty --demo), Pesty window only — capture will be attached before this draft is marked ready for review. The local Mac is currently locked, preventing UI capture.

alvst added 30 commits July 23, 2026 19:33
…ttings-polish

# Conflicts:
#	Sources/Pesty/Settings/Settings.swift
…tings-polish

# Conflicts:
#	Sources/Pesty/Settings/Settings.swift
…-settings-polish

# Conflicts:
#	Sources/Pesty/AppController.swift
#	Sources/Pesty/Settings/Settings.swift
#	Sources/Pesty/Settings/SettingsView.swift
…s-polish

# Conflicts:
#	Sources/Pesty/UI/BarWindowController.swift
…ettings-polish

# Conflicts:
#	Sources/Pesty/Settings/Settings.swift
#	Sources/Pesty/Settings/SettingsView.swift
…tings-polish

# Conflicts:
#	Sources/Pesty/Settings/SettingsView.swift
alvst added 26 commits July 24, 2026 14:38
…view

# Conflicts:
#	Sources/Pesty/Settings/Settings.swift
#	Sources/Pesty/Settings/SettingsView.swift
…all-prs-preview

# Conflicts:
#	Sources/Pesty/AppController.swift
#	Sources/Pesty/Monitor/PasteService.swift
#	Sources/Pesty/Settings/Settings.swift
#	Sources/Pesty/Settings/SettingsView.swift
#	Sources/Pesty/Store/ClipboardStore.swift
#	Sources/Pesty/UI/BarView.swift
…eview

# Conflicts:
#	Sources/Pesty/UI/BarView.swift
…-preview

# Conflicts:
#	Sources/Pesty/UI/BarView.swift
…-preview

# Conflicts:
#	Sources/Pesty/AppController.swift
#	Sources/Pesty/Store/ClipboardStore.swift
#	Sources/Pesty/UI/BarView.swift
# Conflicts:
#	Sources/Pesty/AppController.swift
#	Sources/Pesty/Monitor/PasteService.swift
#	Sources/Pesty/Store/ClipboardStore.swift
#	Sources/Pesty/UI/ClipCardView.swift
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant