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
F18827776: D8431.diff
Fri, Oct 24, 12:03 PM
F18629522: D8431.diff
Sep 16 2025, 9:08 AM
F18596228: D8431.id.diff
Sep 13 2025, 12:05 AM
F18584540: D8431.diff
Sep 11 2025, 12:57 PM
F18090849: D8431.diff
Aug 6 2025, 6:00 PM
F18090138: D8431.id20012.diff
Aug 6 2025, 2:20 PM
F18015822: D8431.id20013.diff
Aug 2 2025, 6:01 PM
F17946301: D8431.diff
Jul 31 2025, 2:11 PM
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