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
Copy link
Copy Markdown
Member

@kiyarose kiyarose commented May 19, 2026

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 Web Open in Cursor 

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

deepsource-io Bot commented May 19, 2026

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 Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Rust May 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.

cursoragent and 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 main May 20, 2026
3 of 5 checks passed
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.

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

2 participants