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

bug/issue 1197 ensure Lit SSR hydration script loads after importmaps #1289

Conversation

thescientist13
Copy link
Member

@thescientist13 thescientist13 commented Oct 25, 2024

Related Issue

related to #1197

Noticed in thescientist13/greenwood-lit-ssr#14 that Chrome started complaining that <script type="module"> tags are not allowed before a <script type="importmap"> (which makes sense)

Screenshot 2024-10-24 at 2 22 12 PM
Screenshot 2024-10-24 at 2 22 32 PM

Summary of Changes

  1. Detect and correctly load hydration script based on development / build time
  2. Add test cases

@thescientist13 thescientist13 force-pushed the bug/issue-1197-hydration-script-tag-loading-before-importmaps branch from a6e2b8f to 948158a Compare November 1, 2024 16:46
@thescientist13 thescientist13 merged commit 5b75cf4 into release/0.30.0 Nov 1, 2024
8 checks passed
@thescientist13 thescientist13 deleted the bug/issue-1197-hydration-script-tag-loading-before-importmaps branch November 1, 2024 17:05
thescientist13 added a commit that referenced this pull request Nov 2, 2024
…#1289)

* ensure Lit hydration script loads after importmaps

* fix attribute selector syntax used in test cases
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alpha.8 bug Something isn't working Plugins Greenwood Plugins SSR v0.30.0
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Lit v3 upgrade and SSR Pages support and compatibility for renderer plugin
1 participant