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
F19151313: D8431.diff
Wed, Dec 10, 10:18 PM
F19057832: D8431.id.diff
Fri, Nov 28, 11:52 PM
F19047037: D8431.diff
Thu, Nov 27, 12:14 PM
F18872726: D8431.id20013.diff
Nov 5 2025, 5:58 AM
F18846311: D8431.diff
Oct 29 2025, 6:27 PM
F18827776: D8431.diff
Oct 24 2025, 12:03 PM
F18629522: D8431.diff
Sep 16 2025, 9:08 AM
F18596228: D8431.id.diff
Sep 13 2025, 12:05 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