Page MenuHomePhabricator
Differential D16920

Prevent media from autoplaying when rendered as a feed story
ClosedPublic
Actions

Authored by epriestley on Nov 22 2016, 10:17 PM.
Tags
None
Referenced Files
F19079272: D16920.id.diff
Dec 1 2025, 8:39 PM
F19073857: D16920.id40734.diff
Dec 1 2025, 3:25 AM
F19067499: D16920.diff
Nov 30 2025, 8:24 AM
F19002109: D16920.diff
Nov 21 2025, 5:04 AM
F18843455: D16920.diff
Oct 28 2025, 10:26 PM
F18841366: D16920.diff
Oct 28 2025, 8:31 AM
F18819849: D16920.id.diff
Oct 22 2025, 10:14 AM
F18764242: D16920.id40735.diff
Oct 7 2025, 5:03 AM
View All Files
Subscribers
None
Tokens
"Party Time" token, awarded by epriestley.

Details

Reviewers
chad
Maniphest Tasks
T11845: Media with "autoplay" should possibly not autoplay in news feed
Commits
rP4dadad53ae5d: Prevent media from autoplaying when rendered as a feed story
Summary

Fixes T11845. Users can still embed a text panel on the home page to give it some ambiance.

Test Plan

Wrote an autoplay video as a comment, saw it in feed. Before change: autoplay. After change: no auto play. On task: still autoplay.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 40734.Nov 22 2016, 10:17 PM
epriestley retitled this revision from to Prevent media from autoplaying when rendered as a feed story.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T11845: Media with "autoplay" should possibly not autoplay in news feed.
epriestley added a comment.Nov 22 2016, 10:18 PM

This is a SAD CHANGE because we HATE FUN.

epriestley awarded a token.Nov 22 2016, 10:18 PM
chad edited edge metadata.Nov 22 2016, 10:19 PM

frowncat

chad accepted this revision.Nov 22 2016, 10:19 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 22 2016, 10:19 PM
Closed by commit rP4dadad53ae5d: Prevent media from autoplaying when rendered as a feed story (authored by epriestley, committed by epriestley). · Explain WhyNov 22 2016, 10:20 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
feed/
story/
2 lines
files/
markup/
6 lines
infrastructure/
markup/
4 lines

Diff 40735

View Options

src/applications/feed/story/PhabricatorFeedStory.php

Loading...
View Options

src/applications/files/markup/PhabricatorEmbedFileRemarkupRule.php

Loading...
View Options

src/infrastructure/markup/PhabricatorMarkupEngine.php

Loading...
Phabricator · Built by Phacility