Page MenuHomePhabricator
Differential D8925

Allow users to provide alt text for image embeds
ClosedPublic
Actions

Authored by epriestley on May 1 2014, 2:08 PM.
Tags
None
Referenced Files
F14352827: D8925.diff
Thu, Dec 19, 2:53 PM
F14334262: D8925.id21178.diff
Wed, Dec 18, 5:08 PM
Unknown Object (File)
Sun, Dec 15, 2:28 PM
Unknown Object (File)
Wed, Dec 4, 8:44 AM
Unknown Object (File)
Wed, Dec 4, 8:44 AM
Unknown Object (File)
Wed, Dec 4, 8:44 AM
Unknown Object (File)
Wed, Dec 4, 8:37 AM
Unknown Object (File)
Tue, Dec 3, 11:58 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
chad
Maniphest Tasks
T4843: Making Phabricator accessible to assistive technologies
Commits
Restricted Diffusion Commit
rPc9dc554cb94f: Allow users to provide alt text for image embeds
Summary

Ref T4843.

  • Add an alt attribute so users can provide alternate text for {Fnnn}.
  • Add an alt attribute to image macros.
Test Plan

Embedded an image with alt and a macro, inspected HTML source to verify the alt attribute was present.

Diff Detail

Repository
rP Phabricator
Branch
aural2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 163
Build 163: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 21178.May 1 2014, 2:08 PM
epriestley retitled this revision from to Allow users to provide alt text for image embeds.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
epriestley added a task: T4843: Making Phabricator accessible to assistive technologies.
Herald added a subscriber: epriestley. · View Herald TranscriptMay 1 2014, 2:08 PM
Harbormaster completed remote builds in B163: Diff 21178.May 1 2014, 2:09 PM
chad accepted this revision.May 1 2014, 2:13 PM
chad edited edge metadata.
chad added inline comments.
src/applications/macro/remarkup/PhabricatorRemarkupRuleMeme.php
39

omg, memes for the blind.

This revision is now accepted and ready to land.May 1 2014, 2:13 PM
epriestley closed this revision.May 1 2014, 2:18 PM
epriestley updated this revision to Diff 21181.

Closed by commit rPc9dc554cb94f (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
files/
remarkup/
5 lines
macro/
remarkup/
7 lines
docs/
user/
userguide/
3 lines

Diff 21178

View Options

src/applications/files/remarkup/PhabricatorRemarkupRuleEmbedFile.php

Loading...
View Options

src/applications/macro/remarkup/PhabricatorRemarkupRuleMeme.php

Loading...
View Options

src/docs/user/userguide/remarkup.diviner

Loading...
Phabricator · Built by Phacility