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
F14037087: D8431.id20013.diff
Sun, Nov 10, 1:45 PM
F14037085: D8431.id20012.diff
Sun, Nov 10, 1:45 PM
F13985577: D8431.id.diff
Sun, Oct 20, 9:26 PM
F13957816: D8431.id20012.diff
Mon, Oct 14, 11:28 AM
Unknown Object (File)
Oct 13 2024, 10:20 AM
Unknown Object (File)
Sep 20 2024, 11:53 AM
Unknown Object (File)
Sep 14 2024, 4:54 PM
Unknown Object (File)
Sep 14 2024, 4:52 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