Page MenuHomePhabricator

Remove old Phrequent propery rendering code and show "Time Spent" in higher precision

Authored by epriestley on Dec 22 2018, 11:57 AM.




Phrequent has two nearly-identical copies of its rendering code: one for old "property event" objects and one for newer "curtain" objects. In the upstream, both trackable object types (tasks and revisions) use curtains, so throw away the old code since it isn't reachable. Third-party trackable objects can update to the curtain UI, but it's unlikely they exist.

Render the remaining curtain UI with more precision, so we show "Time Spent: 2d, 11h, 49m" instead of "Time Spent: 2d".

Test Plan

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline


(This method is largely identical to the removed method, below. It renders into the newer "curtain" sidebar element; the old one rendered directly into the main column's list of properties.)

This revision is now accepted and ready to land.Dec 24 2018, 10:14 PM
This revision was automatically updated to reflect the committed changes.