Skip to content

feat: embed image and video previews in notes - #23

Merged
kiyarose merged 4 commits into
mainfrom
cursor/embed-media-renderer-5158
May 20, 2026
Merged

feat: embed image and video previews in notes#23
kiyarose merged 4 commits into
mainfrom
cursor/embed-media-renderer-5158

Conversation

@kiyarose

@kiyarosekiyarose commented May 19, 2026

Copy link
Copy Markdown
Member

Summary

  • add an embedded media preview panel below the note editor
  • detect image and video URLs in note text and render image/video cards inline
  • enable video playback controls for embedded video links
  • expand Tauri CSP to allow external HTTPS image/video resources to load in embedded renderers

Manual verification (single run)

  • entered image and video URLs in the note field
  • verified embedded image rendering works
  • verified video card rendering and playback interaction attempt in-app

Resolves#20
Resolves#21

Open in WebOpen in Cursor

Co-authored-by: Kiya Rose Ren-Miyakari <kiya.rose@sillylittle.tech>
@deepsource-io

deepsource-ioBot commented May 19, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 0b424d0...92f0104 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall GradeSecurity

Reliability

Complexity

Hygiene

Code Review Summary

AnalyzerStatusUpdated (UTC)Details
RustMay 20, 2026 5:41a.m.Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

cursoragentand others added 2 commits May 19, 2026 22:01
Co-authored-by: Kiya Rose Ren-Miyakari <kiya.rose@sillylittle.tech>
@kiyarose
kiyarose marked this pull request as ready for review May 20, 2026 05:23
@kiyarose
kiyarose marked this pull request as draft May 20, 2026 05:23
This overlay mode with slider, resolves#21
@kiyarose
kiyarose marked this pull request as ready for review May 20, 2026 05:41
@kiyarose
kiyarose enabled auto-merge May 20, 2026 05:41
@kiyarose
kiyarose merged commit df14682 into mainMay 20, 2026
3 of 5 checks passed
Sign up for freeto 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.

feat: Allow Overlaying Pinstick feat: allow image and video support for pinstick

2 participants

@kiyarose@cursoragent