Page MenuHomePhabricator

Fix Maniphest Reports scrolling issues
ClosedPublic

Authored by chad on Feb 24 2015, 5:33 PM.
Tags
None
Referenced Files
F18220911: D11879.id28605.diff
Tue, Aug 19, 4:32 PM
F18208224: D11879.id28604.diff
Mon, Aug 18, 8:20 PM
F18201969: D11879.id.diff
Mon, Aug 18, 9:07 AM
F18195656: D11879.diff
Sun, Aug 17, 4:38 PM
F17733840: D11879.diff
Jul 20 2025, 9:15 AM
F17733618: D11879.id28605.diff
Jul 20 2025, 8:34 AM
F17733609: D11879.id28604.diff
Jul 20 2025, 8:34 AM
Unknown Object (File)
Jun 27 2025, 6:31 AM
Subscribers

Details

Summary

Fixes T7099, also picked some new colors. Raphael can bind the graph to a dom element, which resolved the scrolling issue.

Test Plan

Tested scrolling on my laptop, desktop. Seems resolved.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Fix Maniphest reports.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad retitled this revision from Fix Maniphest reports to Fix Maniphest Reports scrolling issues.Feb 24 2015, 5:34 PM
btrahan edited edge metadata.
btrahan added inline comments.
webroot/rsrc/css/application/maniphest/report.css
15–16

looks like we lost some CSS rules; just double checking that was intended

webroot/rsrc/js/application/maniphest/behavior-line-chart.js
16

this is a wacky constructor IMO - weird to see two parameters can instead be a single parameter of a different type - but hey we didn't write it!

This revision is now accepted and ready to land.Feb 24 2015, 5:36 PM
webroot/rsrc/css/application/maniphest/report.css
15–16

These are pretty well denoted by "Week of" that they don't also need a color.

webroot/rsrc/js/application/maniphest/behavior-line-chart.js
16
This revision was automatically updated to reflect the committed changes.