Skip to content

Add "text" destination type, for JavaScript text imports#1898

Merged
annevk merged 2 commits intowhatwg:mainfrom
eemeli:import-text
Apr 1, 2026
Merged

Add "text" destination type, for JavaScript text imports#1898
annevk merged 2 commits intowhatwg:mainfrom
eemeli:import-text

Conversation

@eemeli
Copy link
Copy Markdown
Member

@eemeli eemeli commented Dec 17, 2025

This change is to support the changes in whatwg/html#11933, by adding text as a destination type.


Preview | Diff

Copy link
Copy Markdown
Member

@annevk annevk left a comment

Choose a reason for hiding this comment

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

This looks reasonable to me, modulo table nit.

Comment thread fetch.bs
@styfle
Copy link
Copy Markdown
Contributor

styfle commented Jan 25, 2026

@eemeli @annevk Can this be merged now? I'm going to make a similar PR for import bytes and I'm hoping to avoid merge conflicts.

@zcorpan
Copy link
Copy Markdown
Member

zcorpan commented Mar 31, 2026

I guess implementation bugs here are the same as in whatwg/html#11933

@eemeli
Copy link
Copy Markdown
Member Author

eemeli commented Mar 31, 2026

I guess implementation bugs here are the same as in whatwg/html#11933

Yes; I've updated the description to match.

zcorpan pushed a commit to web-platform-tests/wpt that referenced this pull request Apr 1, 2026
@annevk annevk merged commit dfdd250 into whatwg:main Apr 1, 2026
2 checks passed
@eemeli eemeli deleted the import-text branch April 1, 2026 18:32
lando-worker Bot pushed a commit to mozilla-firefox/firefox that referenced this pull request Apr 17, 2026
Automatic update from web-platform-tests
Add tests for text modules

This is to support the spec changes in:

- https://github.com/tc39/proposal-import-text
- whatwg/html#11933
- whatwg/fetch#1898
- w3c/webappsec-csp#794
--

wpt-commits: 5419f98d72f4b1ae49980581a6a66710d6a3dee7
wpt-pr: 56812
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants