Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Export the two CSP directives #545

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Conversation

lukewarlow
Copy link
Member

@lukewarlow lukewarlow commented Sep 9, 2024

Closes #538


Preview | Diff

@lukewarlow
Copy link
Member Author

lukewarlow commented Sep 9, 2024

@robbiemc I think this should be enough for your requirements?

Apologies for the delay in addressing this.

Copy link
Contributor

@fred-wang fred-wang left a comment

Choose a reason for hiding this comment

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

Not super familiar with bikeshed, but that looks correct per https://speced.github.io/bikeshed/#dfn-export

It seems @robbiemc already merged WICG/isolated-web-apps#43 with a <pre class="anchors"> tweak, but this may still be useful anyway.

@koto koto merged commit e5717b9 into w3c:main Oct 28, 2024
3 checks passed
github-actions bot added a commit that referenced this pull request Oct 28, 2024
SHA: e5717b9
Reason: push, by koto

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
robbiemc added a commit to robbiemc/isolated-web-apps that referenced this pull request Oct 28, 2024
w3c/trusted-types#545 exports the
require-trusted-types-for-directive definition so the manually
defined anchor section isn't needed anymore.
@robbiemc
Copy link

Sorry, I missed your original question about this!

This does indeed make it linkable, and I sent WICG/isolated-web-apps#55 to remove the anchor definition now that the link works. Thanks!

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.

Export the 'require-trusted-types-for' dfn
4 participants