Skip to content

[xExtension-YouTube] Add optional responsive player sizing#488

Merged
Alkarex merged 1 commit into
FreshRSS:mainfrom
JamBalaya56562:fix/youtube-player-autosize
Jul 12, 2026
Merged

[xExtension-YouTube] Add optional responsive player sizing#488
Alkarex merged 1 commit into
FreshRSS:mainfrom
JamBalaya56562:fix/youtube-player-autosize

Conversation

@JamBalaya56562

Copy link
Copy Markdown
Contributor

Summary

This takes over the work from #291 and applies its review feedback to the current implementation.

Behavior

When automatic sizing is enabled, the player uses the existing youtube-plugin-video class together with yt-player-autosize, which applies a full-width 16:9 aspect ratio. When it is disabled or unset, the configured width and height attributes are preserved.

Validation

  • npm run eslint
  • npx stylelint "**/*.css"
  • npm run markdownlint
  • reviewed the generated iframe branches for YouTube, youtube-nocookie, PeerTube, and original-content handling

PHPStan and PHPCS were not run locally because PHP and make are unavailable on PATH and Docker Desktop could not start. They are expected to run in GitHub Actions.

@JamBalaya56562 JamBalaya56562 force-pushed the fix/youtube-player-autosize branch 2 times, most recently from 75b3b4b to 8f413eb Compare July 11, 2026 18:43
@JamBalaya56562 JamBalaya56562 force-pushed the fix/youtube-player-autosize branch from 8f413eb to 29b02a4 Compare July 12, 2026 02:19
@Alkarex Alkarex merged commit 05f64f1 into FreshRSS:main Jul 12, 2026
1 check passed
@JamBalaya56562 JamBalaya56562 deleted the fix/youtube-player-autosize branch July 12, 2026 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants