Skip to content

Enabled clipboard access for copy links#1087

Open
girishpanchal30 wants to merge 1 commit into
developmentfrom
bugfix/1086
Open

Enabled clipboard access for copy links#1087
girishpanchal30 wants to merge 1 commit into
developmentfrom
bugfix/1086

Conversation

@girishpanchal30

Copy link
Copy Markdown
Contributor

All Submissions:

Changes proposed in this Pull Request:

Enabled clipboard access for the Optimole DAM iframe to allow copy functionality.

Closes #1086

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a regression (issue #1086) where the "Copy Link" button inside the Optimole Cloud Library DAM integration silently failed to copy asset URLs to the clipboard. Because the DAM UI is rendered in a cross-origin iframe (https://dashboard.optimole.com/dam), the Clipboard API is blocked unless the embedding page explicitly delegates the permission. The fix adds the allow="clipboard-read; clipboard-write" attribute to the DAM iframe so clipboard access is granted in both the standalone Cloud Library admin page and the WordPress media modal, which share the same render_dashboard_page() markup.

Changes:

  • Added allow="clipboard-read; clipboard-write" to the #om-dam iframe to enable Clipboard API access for the cross-origin DAM UI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pirate-bot

Copy link
Copy Markdown
Collaborator

Plugin build for e95cf39 is ready 🛎️!

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.

4 participants