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
F18098527: D16920.id40735.diff
Fri, Aug 8, 8:45 PM
F18093111: D16920.id40734.diff
Thu, Aug 7, 3:43 PM
F18092966: D16920.diff
Thu, Aug 7, 2:34 PM
F18086516: D16920.id.diff
Tue, Aug 5, 10:17 PM
F17915556: D16920.diff
Tue, Jul 29, 8:48 PM
Unknown Object (File)
Jun 20 2025, 9:10 AM
Unknown Object (File)
Jun 20 2025, 7:00 AM
Unknown Object (File)
May 29 2025, 8:48 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