Page MenuHomePhabricator
Differential D8431

Include Source Sans on-demand with Celerity
ClosedPublic
Actions

Authored by epriestley on Mar 6 2014, 7:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 14, 4:54 PM
Unknown Object (File)
Sat, Sep 14, 4:52 PM
Unknown Object (File)
Sat, Sep 14, 4:52 PM
Unknown Object (File)
Sat, Sep 14, 4:52 PM
Unknown Object (File)
Tue, Sep 3, 11:04 PM
Unknown Object (File)
Mon, Aug 26, 9:27 AM
Unknown Object (File)
Mon, Aug 26, 6:49 AM
Unknown Object (File)
Mon, Aug 26, 1:53 AM
View All Files
Subscribers
aran

Details

Reviewers
chad
Commits
Restricted Diffusion Commit
rP270929dd6b4e: Include Source Sans on-demand with Celerity
Summary

Unwinds the mess I made in D8422 / D8430:

  • Remove 'fonts', since individual fonts can be included via Celerity now.
  • Include Source Sans from the local source when a document uses it as a fontkit.
Test Plan

Browsed Diviner, saw Source Sans.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

chad accepted this revision.Mar 6 2014, 7:39 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 6 2014, 7:39 PM
epriestley closed this revision.Mar 6 2014, 7:39 PM

Closed by commit rP270929dd6b4e (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
base/
controller/
4 lines
diviner/
controller/
1 line
1 line
1 line
view/
page/
21 lines
phui/
6 lines

Diff 20013

View Options

src/applications/base/controller/PhabricatorController.php

Loading...
View Options

src/applications/diviner/controller/DivinerAtomController.php

Loading...
View Options

src/applications/diviner/controller/DivinerBookController.php

Loading...
View Options

src/applications/diviner/controller/DivinerFindController.php

Loading...
View Options

src/view/page/PhabricatorStandardPageView.php

Loading...
View Options

src/view/phui/PHUIDocumentView.php

Loading...
Phabricator · Built by Phacility