Deprecated scrolling="no" iframe attribute used instead of CSS #8

New issue

Closed

opened 2026-07-28 21:01:57 +00:00 by paulloth1 · 1 comment

paulloth1 commented

2026-07-28 21:01:57 +00:00

(Migrated from github.com)

Problem

`backend/slides/media.py` renders URL-embed iframes with the legacy HTML attribute `scrolling="no"`. This attribute was dropped from the HTML5 spec (it's obsolete/non-conforming) and only still works because browsers keep it for backwards compatibility.

Suggested fix

Replace with the standards-compliant CSS equivalent, e.g. `overflow: hidden` on the iframe (or a wrapping container), which achieves the same "no scrollbars" effect without relying on deprecated markup.

## Problem \`backend/slides/media.py\` renders URL-embed iframes with the legacy HTML attribute \`scrolling="no"\`. This attribute was dropped from the HTML5 spec (it's obsolete/non-conforming) and only still works because browsers keep it for backwards compatibility. ## Suggested fix Replace with the standards-compliant CSS equivalent, e.g. \`overflow: hidden\` on the iframe (or a wrapping container), which achieves the same "no scrollbars" effect without relying on deprecated markup.

paulloth1 commented

2026-07-28 21:23:00 +00:00

(Migrated from github.com)

Fixed in a841b02 (merged to main at 103cdf2): removed scrolling="no" entirely; iframes now always carry an inline style with overflow:hidden instead.

Fixed in a841b02 (merged to main at 103cdf2): removed scrolling="no" entirely; iframes now always carry an inline style with overflow:hidden instead.