Skip to content

Conversation

@sra405
Copy link
Contributor

@sra405 sra405 commented Apr 17, 2024

As part of project migration from trinket to the code editor some additional functionality is required. This PR enables iframes to be used from the same approved external sources list as used for href's and other src attributes. By expanding this to allow for self references (i.e. src with a value the same as the PUBLIC_URL) users can embed other projects they (or we) own in iframes into their projects

@sra405 sra405 temporarily deployed to previews/enable-external-iframes April 17, 2024 11:26 — with GitHub Actions Inactive
@github-actions
Copy link

@github-actions
Copy link

@sra405
Copy link
Contributor Author

sra405 commented Apr 17, 2024

Allows iframes to be embedded if they're on the same domain as the editor or on rpf.io
image

Copy link
Contributor

@loiswells97 loiswells97 left a comment

Choose a reason for hiding this comment

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

Looks great 👍

@sra405 sra405 merged commit 5197fac into main Apr 18, 2024
@sra405 sra405 deleted the enable-external-iframes branch April 18, 2024 08:16
floehopper added a commit that referenced this pull request May 9, 2024
### Added

- Support to enable embedding iframes in HTML projects from in-house domains (#985)
- Dispatch event when project identifier changes, e.g. after project is remixed (#2830)
- Add `load_remix_disabled` attribute to web component (#992)

### Changed

- Invalidate cached project when project is remixed (#1003)

### Fixed

- Unit tests for `pyodide` runner (#976)
- Remove broken `format` script (#991)
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.

3 participants