Page MenuHomePhabricator
Differential D17878

Make a handful of minor Slowvote behaviors more consistent with other applications
ClosedPublic
Actions

Authored by epriestley on May 14 2017, 8:51 PM.
Tags
None
Referenced Files
F15511241: D17878.id42998.diff
Thu, Apr 17, 12:44 AM
F15470596: D17878.id43004.diff
Fri, Apr 4, 11:38 PM
F15439976: D17878.id43004.diff
Mar 26 2025, 10:37 AM
F15436336: D17878.id43000.diff
Mar 25 2025, 1:31 PM
F15434994: D17878.id.diff
Mar 25 2025, 6:18 AM
F15430207: D17878.diff
Mar 24 2025, 6:28 AM
F15429591: D17878.id43000.diff
Mar 24 2025, 3:03 AM
Unknown Object (File)
Mar 1 2025, 11:38 PM
View All Files
Subscribers
None

Details

Reviewers
chad
amckinley
Maniphest Tasks
T12685: Modular Transactions Migrations Errata
Commits
rPc7a642255997: Make a handful of minor Slowvote behaviors more consistent with other…
Summary

Ref T12685.

  • Better icon/color/label consistency.
  • Make "0" a valid response.
  • Fix a bug where creating a poll with Quicksand enabled led to bad times (form submitted back to the search screen).
Test Plan

Screen Shot 2017-05-14 at 1.51.04 PM.png (1×1 px, 154 KB)

Diff Detail

Repository
rP Phabricator
Branch
err4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16923
Build 22615: Run Core Tests
Build 22614: arc lint + arc unit

Event Timeline

epriestley created this revision.May 14 2017, 8:51 PM
Harbormaster completed remote builds in B16922: Diff 42998.May 14 2017, 8:52 PM
epriestley updated this revision to Diff 43000.May 14 2017, 8:54 PM
  • Bonus! Change creation string from "created this object" to "created this poll".
Harbormaster completed remote builds in B16923: Diff 43000.May 14 2017, 8:55 PM
chad accepted this revision.May 14 2017, 8:56 PM
This revision is now accepted and ready to land.May 14 2017, 8:56 PM
Closed by commit rPc7a642255997: Make a handful of minor Slowvote behaviors more consistent with other… (authored by epriestley, committed by epriestley). · Explain WhyMay 14 2017, 9:01 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 43000

View Options

src/applications/slowvote/controller/PhabricatorSlowvoteEditController.php

Loading...
View Options

src/applications/slowvote/controller/PhabricatorSlowvotePollController.php

Loading...
View Options

src/applications/slowvote/editor/PhabricatorSlowvoteEditor.php

Loading...
View Options

src/applications/slowvote/query/PhabricatorSlowvoteSearchEngine.php

Loading...
View Options

src/applications/slowvote/storage/PhabricatorSlowvotePoll.php

Loading...
Phabricator · Built by Phacility