Page MenuHomePhabricator

Use proper file icons in lightbox
ClosedPublic

Authored by chad on Nov 28 2016, 7:16 PM.
Tags
None
Referenced Files
F13153223: D16957.id.diff
Sun, May 5, 1:29 AM
Unknown Object (File)
Fri, May 3, 3:35 AM
Unknown Object (File)
Thu, May 2, 9:56 PM
Unknown Object (File)
Tue, Apr 30, 10:31 AM
Unknown Object (File)
Mon, Apr 29, 3:27 PM
Unknown Object (File)
Mon, Apr 29, 5:51 AM
Unknown Object (File)
Fri, Apr 26, 5:47 PM
Unknown Object (File)
Wed, Apr 24, 11:26 PM
Subscribers

Details

Summary

Ref T3612. Passes in file size and file icon for non-images.

Test Plan

Review a PDF and PSD in a lightbox.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Use proper file icons in lightbox.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T3612: Lightbox v2.

Add "services/src/" to your "load-libraries" config to clear that test failure, I think?

epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 28 2016, 7:19 PM

I refuse to conform to your "rules".

This revision was automatically updated to reflect the committed changes.