Page MenuHomePhabricator

Add a basic progress bar for milestones
ClosedPublic

Authored by epriestley on Feb 9 2016, 1:36 AM.
Tags
None
Referenced Files
F14052279: D15221.diff
Fri, Nov 15, 8:01 AM
F14040750: D15221.diff
Mon, Nov 11, 1:01 PM
F14024885: D15221.diff
Thu, Nov 7, 1:06 PM
F13993860: D15221.id36744.diff
Wed, Oct 23, 1:55 AM
F13988104: D15221.id36741.diff
Mon, Oct 21, 1:08 PM
F13966552: D15221.id36744.diff
Oct 16 2024, 8:29 AM
F13964920: D15221.diff
Oct 15 2024, 10:43 PM
Unknown Object (File)
Oct 8 2024, 10:52 AM
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T4427: Add formal task points
Commits
Restricted Diffusion Commit
rP0782652a8059: Add a basic progress bar for milestones
Summary

Ref T4427. This kind of works.

Test Plan

Screen Shot 2016-02-08 at 5.32.55 PM.png (1×1 px, 110 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Add a basic progress bar for milestones.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
src/applications/maniphest/editor/ManiphestEditEngine.php
310

This stuff isn't directly related, but if you inline-edit a task which only belongs to a column implicitly we currently error out.

This fixes it, but overfetches.

All the related code needs changes soon anyway to accommodate a bunch of live-edit bugs so I'll deal with it more surgically in future diffs.

Do you think this needs to be a separate panel? I assumed it would just be baked into the profile panel.

Seems somewhat useful to me to be able to hide/reorder it separately from the other items. It might also eventually get config, maybe.

(I also think clicking it might take you to some sort of charts eventually.)

My concern is mostly around people being able to make bad UI choices though unlimited customization. I'm fine giving this a whirl, but we probably should think about some sort of "global" settings for profile menus, so people at least have some way of keeping things consistent. Consistently bad is at least better than randomly bad and confusing.

chad edited edge metadata.
This revision is now accepted and ready to land.Feb 9 2016, 2:46 AM
This revision was automatically updated to reflect the committed changes.