Tecnoblog's Experience With Discourse Comments

Falco · April 16, 2026, 5:24pm

Oh that is now widely supported across browsers? TIL! Will change is ASAP.

github.com/discourse/discourse

PERF: Use native iframe lazy loading for full app embeds (#39320)

main ← perf-native-iframe-lazy-loading

merged 05:41PM - 16 Apr 26 UTC

xfalcox

+4 -25

## Summary Replaces the `IntersectionObserver` implementation from #39299 with …the native `loading="lazy"` attribute on the iframe. All supported browsers (Chrome 77+, Safari 16.4+, Firefox 121+) handle this, and the browser-determined threshold adapts to connection speed — generally smarter than a fixed pixel margin. - Drops ~20 lines of JS and the feature-detection fallback - Removes the `lazyLoadMargin` option (no longer configurable — browser decides) - `lazyLoad: false` still opts out by setting `loading="eager"` - Admin embedding snippet updated to drop the `lazyLoadMargin` line Shipped two days ago in #39299, so no external consumers should depend on `lazyLoadMargin` yet. ## Test plan - [ ] Full app embed below the fold — verify iframe defers loading until scrolling near it - [ ] Full app embed above the fold — verify it loads immediately - [ ] `lazyLoad: false` — verify immediate loading, same as pre-#39299 behavior - [ ] Standard (non-full-app) embed — verify unchanged behavior

Topic		Replies	Views
Embed Discourse comments on another website via Javascript Integrations embedding , how-to	123	302099	October 27, 2025
Discourse Comments stuck on "Loading..." Support	22	2105	September 18, 2024
Embed Discourse comments with on-page commenting? Feature	32	7995	January 19, 2019
How to automatically adjust iframe height for embedded wordpress posts WordPress	18	4927	June 26, 2025
Cannot get embedding to work Support	11	2574	August 31, 2021

Tecnoblog's Experience With Discourse Comments

Related topics