Skip to content

Follow-up to #8: Add floating inline clip previews#28

Draft
alvst wants to merge 5 commits into
momenbasel:mainfrom
alvst:pr-10-link-preview-popover
Draft

Follow-up to #8: Add floating inline clip previews#28
alvst wants to merge 5 commits into
momenbasel:mainfrom
alvst:pr-10-link-preview-popover

Conversation

@alvst

@alvst alvst commented Jul 24, 2026

Copy link
Copy Markdown

Dependency

This draft builds on #8 (Add configurable clip previews). Its current diff intentionally includes #8 because GitHub cannot use a branch in this fork as the base of an upstream PR.

After #8 merges, this branch will be rebased onto upstream main; this PR will then show only the floating-preview follow-up.

What changed

  • Move Inline Pesty previews into a separate floating panel anchored to the selected clip, so the Pesty bar remains at its normal height.
  • Give the preview a clean document-style header, canvas, metadata footer, pointer, and spring entrance animation.
  • Present Finder-copied image files as images throughout the UI and Quick Look, while preserving their original file-paste behavior.
  • Keep rich link previews with favicon/title metadata and an Open in Safari action.

Why

The first inline-preview implementation resized and rendered inside the Pesty bar, which made the quick view feel like part of the strip rather than an independent preview. The floating panel gives the preview its own visual space and keeps navigation stable.

Validation

  • swift build -c release --arch arm64 --arch x86_64
  • swift build -c release --arch arm64 --arch x86_64 -Xswiftc -DMAS
  • Packaged and tested locally on Apple Silicon.
  • Intel hardware testing was not available; the x86_64 build completed successfully.

Screenshots

Demo mode; Pesty window only.

Floating Pesty text preview

@alvst alvst changed the title Follow-up to #8: Add anchored link preview popover Follow-up to #8: Add floating inline clip previews Jul 24, 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.

1 participant