Page MenuHomePhabricator
Differential D20501

Provide chart function labels over the wire instead of making them up
ClosedPublic
Actions

Authored by epriestley on May 8 2019, 2:22 PM.
Tags
None
Referenced Files
F18278779: D20501.id.diff
Fri, Aug 22, 8:56 PM
F18250855: D20501.diff
Thu, Aug 21, 5:11 PM
F18112438: D20501.id49001.diff
Tue, Aug 12, 5:04 PM
F18106409: D20501.id48894.diff
Sun, Aug 10, 10:34 PM
F18106094: D20501.diff
Sun, Aug 10, 9:10 PM
F18105778: D20501.id49001.diff
Sun, Aug 10, 7:34 PM
F18105768: D20501.id.diff
Sun, Aug 10, 7:32 PM
F18100497: D20501.id.diff
Sat, Aug 9, 11:55 AM
View All Files
Subscribers
yelirekim

Details

Reviewers
amckinley
Maniphest Tasks
T13279: Build Charting for Facts
Commits
rPa80426b339c4: Provide chart function labels over the wire instead of making them up
Summary

Ref T13279. Makes charts incrementally more useful by allowing the server to provide labels and colors for functions.

Test Plan

Screen Shot 2019-05-08 at 7.16.14 AM.png (1×1 px, 193 KB)

Diff Detail

Repository
rP Phabricator
Branch
chart24
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22767
Build 31221: Run Core Tests
Build 31220: arc lint + arc unit

Event Timeline

epriestley created this revision.May 8 2019, 2:22 PM
Herald added a subscriber: yelirekim. · View Herald TranscriptMay 8 2019, 2:22 PM
Harbormaster completed remote builds in B22767: Diff 48894.May 8 2019, 2:24 PM
epriestley requested review of this revision.May 8 2019, 2:24 PM
amckinley accepted this revision.May 21 2019, 9:38 PM
amckinley added inline comments.
src/applications/fact/chart/PhabricatorChartStackedAreaDataset.php
138

Intended placeholder?

This revision is now accepted and ready to land.May 21 2019, 9:38 PM
epriestley added inline comments.May 22 2019, 12:22 PM
src/applications/fact/chart/PhabricatorChartStackedAreaDataset.php
138

Yeah, I think this gets a real name in a followup.

Closed by commit rPa80426b339c4: Provide chart function labels over the wire instead of making them up (authored by epriestley). · Explain WhyMay 22 2019, 12:23 PM
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPa80426b339c4: Provide chart function labels over the wire instead of making them up.
epriestley mentioned this in rPa80426b339c4: Provide chart function labels over the wire instead of making them up.

Revision Contents
Changeset List

PathSize
resources/
celerity/
src/
applications/
fact/
chart/
17 lines
56 lines
15 lines
webroot/
rsrc/
js/
application/
fact/
13 lines
10 lines
35 lines

Diff 48894

View Options

resources/celerity/map.php

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/fact/chart/PhabricatorChartFunction.php

Loading...
View Options

src/applications/fact/chart/PhabricatorChartFunctionLabel.php

Loading...
View Options

src/applications/fact/chart/PhabricatorChartStackedAreaDataset.php

Loading...
View Options

webroot/rsrc/js/application/fact/Chart.js

Loading...
View Options

webroot/rsrc/js/application/fact/ChartCurtainView.js

Loading...
View Options

webroot/rsrc/js/application/fact/ChartFunctionLabel.js

Loading...
Phabricator · Built by Phacility