Skip to content

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

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

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
Contributor

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 24 commits July 24, 2026 14:45
…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
@momenbasel

Copy link
Copy Markdown
Owner

Closing as part of the stacked chain - full reasoning and the re-cut plan are in #55 (#55 (comment)). Please keep this branch; the sequencing in that comment brings this work back rebased on current main.

@momenbasel momenbasel closed this Aug 11, 2026
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.

2 participants