Menu ▾ ▴

#263 Expand context-menu coverage for media and attachment links

closed
nobody
6 days ago
2026-08-28
Anonymous
No

Originally created by: kristofferR

Problem

Carrier's custom context menu can miss useful media targets when Messenger does not expose them as a conventional image element. Video, CSS background media, and attachment links also need clearer actions.

Overly broad detection would be risky because Messenger contains many avatars, emoji, sprites, and internal controls.

Goal

Provide dependable, narrowly targeted context actions for real message media and safe attachments.

Design

  • Recognize actual message images even when alt text is absent, using currentSrc and structural context.
  • Add Open in browser for supported message video targets.
  • Recognize inline CSS background images only when they belong to message media, not avatars, emoji, reactions, icons, or application chrome.
  • Add Download File for attachment links by routing through the safe download policy in [#252].
  • Preserve native selection, copy, share, accessibility, and link behavior where Carrier has no better action.
  • Prefer original media URLs or native blob handling over canvas re-encoding, which can fail across origins or degrade quality.
  • Avoid broad DOM scans or copying private message content into long-lived state.

Verification

  • Test image, video, GIF, sticker, emoji, avatar, reaction, background-image, blob, and attachment-link fixtures.
  • Confirm irrelevant targets retain normal context-menu behavior.
  • Confirm executable or unsafe attachment types remain blocked.
  • Manually test current Messenger markup on all supported platforms.
  • [#89] and [#132] cover prior media and context-menu work.
  • [#252] owns the safe attachment-download backend.

Related

Tickets: #132
Tickets: #252
Tickets: #264
Tickets: #290
Tickets: #367
Tickets: #409
Tickets: #89

Discussion

  • Anonymous

    Anonymous - 6 days ago

    Originally posted by: kristofferR

    [!NOTE]
    🤖 Claude Opus 5.5 responding on behalf of Kristoffer

    Closing as not planned. The right-click menu already covers message images, videos (download, share, copy address) and links (inject/src/messenger/features/context-menu.ts). What is left, recognizing CSS background media and adding "Open in browser" for videos, adds little and risks matching avatars, emoji and app chrome. Downloading file attachments belongs to [#252].

     

    Related

    Tickets: #252

  • Anonymous

    Anonymous - 6 days ago

    Ticket changed by: kristofferR

    • status: open --> closed
     

Log in to post a comment.