Page MenuHomePhabricator

Use proper file icons in lightbox
ClosedPublic

Authored by chad on Nov 28 2016, 7:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 17, 5:26 AM
Unknown Object (File)
Wed, Dec 11, 10:15 AM
Unknown Object (File)
Fri, Dec 6, 12:57 PM
Unknown Object (File)
Fri, Dec 6, 12:13 AM
Unknown Object (File)
Tue, Dec 3, 11:40 AM
Unknown Object (File)
Wed, Nov 27, 4:05 AM
Unknown Object (File)
Tue, Nov 26, 1:55 AM
Unknown Object (File)
Sat, Nov 23, 3:46 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
Branch
file-lightbox-icon (branched from master)
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 14698
Build 19202: Run Core Tests
Build 19201: arc lint + arc unit

Unit TestsFailed

TimeTest
58 msPhabricatorLocalTimeTestCase::Unknown Unit Message ("")
EXCEPTION (PhutilMissingSymbolException): Failed to load class or interface 'PhacilityServices': the class or interface 'PhacilityServices' is not defined in the library map for any loaded phutil library. If you are not a developer, this almost always means that a library is out of date. For example, you may have upgraded `phabricator` without upgrading `libphutil`, or vice versa. It might also mean that you need to restart Apache or PHP-FPM. Make sure all libraries are up to date and all services have been restarted.
1 msAlmanacNamesTestCase::Unknown Unit Message ("")
30 assertions passed.
0 msAlmanacServiceTypeTestCase::Unknown Unit Message ("")
1 assertion passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
4 assertions passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
3 assertions passed.
View Full Test Results (1 Failed · 343 Passed)

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.