Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Jan 14, 2026

In some cases this might help to reduce scripts loaded on page load.

@susnux susnux added this to the v4.0.0 milestone Jan 14, 2026
@susnux susnux requested review from CarlSchwan and artonge January 14, 2026 15:59
@susnux susnux added type: enhancement 🚀 New feature or request 3. to review 3️⃣ Waiting for reviews labels Jan 14, 2026
@codecov
Copy link

codecov bot commented Jan 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.16%. Comparing base (ff2cffd) to head (6bf0138).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1459      +/-   ##
==========================================
+ Coverage   86.12%   86.16%   +0.03%     
==========================================
  Files          27       27              
  Lines         692      694       +2     
  Branches      197      198       +1     
==========================================
+ Hits          596      598       +2     
  Misses         82       82              
  Partials       14       14              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ShGKme
Copy link
Contributor

ShGKme commented Jan 14, 2026

Does files have a spinner/placeholder during loading?

@susnux
Copy link
Contributor Author

susnux commented Jan 14, 2026

Does files have a spinner/placeholder during loading?

Yes it has a loading spinner until whenDefined resolves.

I already use it in a hacky way: Register the web component when enabled() was called and the tab will return true 😅

@susnux susnux force-pushed the feat/postpone-register branch 2 times, most recently from fbd7a69 to 29a885e Compare January 15, 2026 12:06
@susnux susnux requested a review from ShGKme January 15, 2026 12:07
@ShGKme
Copy link
Contributor

ShGKme commented Jan 15, 2026

Unrelated: why is enabled required?

@susnux susnux force-pushed the feat/postpone-register branch 2 times, most recently from e12bc3b to 485f099 Compare January 15, 2026 13:42
susnux and others added 2 commits January 15, 2026 15:27
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de>
Co-authored-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the feat/postpone-register branch from 485f099 to 6bf0138 Compare January 15, 2026 14:27
@susnux susnux merged commit 7c2ff4a into main Jan 15, 2026
12 checks passed
@susnux susnux deleted the feat/postpone-register branch January 15, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review 3️⃣ Waiting for reviews type: enhancement 🚀 New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants