Page MenuHomePhabricator
Differential D20500

Label important data on charts
ClosedPublic
Actions

Authored by epriestley on May 8 2019, 1:55 PM.
Tags
None
Referenced Files
F13282549: D20500.diff
Sun, Jun 2, 12:13 PM
F13268803: D20500.diff
Wed, May 29, 5:23 AM
F13260214: D20500.id49000.diff
Sun, May 26, 11:37 PM
F13256788: D20500.diff
Sat, May 25, 5:31 PM
F13233651: D20500.diff
Tue, May 21, 2:28 AM
F13227185: D20500.id48893.diff
Mon, May 20, 1:32 AM
F13226418: D20500.id48893.diff
Sun, May 19, 8:37 PM
F13213721: D20500.diff
Fri, May 17, 8:02 AM
View All Files
Subscribers
yelirekim

Details

Reviewers
amckinley
Maniphest Tasks
T13279: Build Charting for Facts
Commits
rPc6052b41a604: Label important data on charts
Summary

Ref T13279. Adds client-side support for rendering function labels on charts, then labels every function as important data.

Works okay on mobile, although I'm not planning to target mobile terribly heavily for v0.

Test Plan

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

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 8 2019, 1:55 PM
Herald added a subscriber: yelirekim. · View Herald TranscriptMay 8 2019, 1:55 PM
epriestley added a comment.May 8 2019, 1:56 PM

I'm leaning toward putting a fair amount of this rendering in JS since I anticipate a lot of JS nonsense later on (hover/click lines, areas, data points, function labels, etc) and all the SVG rendering is somewhat-necessarily in JS anyway.

Harbormaster completed remote builds in B22766: Diff 48893.May 8 2019, 1:57 PM
epriestley requested review of this revision.May 8 2019, 1:57 PM
amckinley accepted this revision.May 21 2019, 9:36 PM
This revision is now accepted and ready to land.May 21 2019, 9:36 PM
Closed by commit rPc6052b41a604: Label important data on charts (authored by epriestley). · Explain WhyMay 22 2019, 12:21 PM
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPc6052b41a604: Label important data on charts.
epriestley mentioned this in rPc6052b41a604: Label important data on charts.

Revision Contents
Changeset List

PathSize
resources/
celerity/
src/
applications/
fact/
engine/
6 lines
webroot/
rsrc/
css/
phui/
41 lines
js/
application/
fact/
79 lines
85 lines

Diff 49000

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/fact/engine/PhabricatorChartRenderingEngine.php

Loading...
View Options

webroot/rsrc/css/phui/phui-chart.css

Loading...
View Options

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

Loading...
View Options

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

Loading...
Phabricator · Built by Phacility