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
F15532069: D8431.diff
Wed, Apr 23, 2:23 PM
F15514988: D8431.diff
Fri, Apr 18, 7:11 AM
F15490217: D8431.id20013.diff
Fri, Apr 11, 3:17 PM
F15489983: D8431.id20013.diff
Fri, Apr 11, 2:00 PM
F15485675: D8431.diff
Thu, Apr 10, 1:42 AM
F15434552: D8431.id20012.diff
Tue, Mar 25, 4:23 AM
F15424482: D8431.id20012.diff
Mar 22 2025, 10:38 PM
F15400097: D8431.id.diff
Mar 17 2025, 9:40 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

Repository
rP Phabricator
Branch
divfont
Lint
Lint Passed
Unit
Tests Passed

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 20012

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