Page MenuHomePhabricator

Fix two minor points UI issues
ClosedPublic

Authored by epriestley on Feb 9 2016, 12:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 12, 11:56 PM
Unknown Object (File)
Thu, Dec 12, 3:32 AM
Unknown Object (File)
Mon, Dec 9, 2:12 AM
Unknown Object (File)
Sun, Dec 8, 7:40 PM
Unknown Object (File)
Wed, Nov 27, 11:05 PM
Unknown Object (File)
Sun, Nov 24, 4:21 AM
Unknown Object (File)
Oct 17 2024, 10:21 PM
Unknown Object (File)
Oct 1 2024, 9:01 AM
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T4427: Add formal task points
Commits
Restricted Diffusion Commit
rPaa6c99384879: Fix two minor points UI issues
Summary

Ref T4427.

  • When points are configured, show them on the task detail page (just a simple property, at least for now).
  • Typecast points better to avoid "joe changed points from 12 to 12." beacuse we compare the stored value (as a string) to the new value (as a double).
Test Plan
  • Saw points on detail view.
  • Created task with points, then edited it without touching points. No more spurious "changed from 12 to 12" transaction.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Fix two minor points UI issues.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 9 2016, 2:25 PM
This revision was automatically updated to reflect the committed changes.