Page MenuHomePhabricator
Differential D15355

Slightly improve Buildable list in Harbormaster
ClosedPublic
Actions

Authored by epriestley on Feb 26 2016, 8:23 PM.
Tags
None
Referenced Files
F15469204: D15355.id37031.diff
Fri, Apr 4, 10:34 AM
F15451870: D15355.id37031.diff
Sat, Mar 29, 12:12 AM
F15398860: D15355.diff
Mon, Mar 17, 1:51 AM
F15388318: D15355.id37027.diff
Sat, Mar 15, 3:39 AM
F15336243: D15355.id37031.diff
Sat, Mar 8, 6:32 PM
F15336242: D15355.id37027.diff
Sat, Mar 8, 6:32 PM
F15336241: D15355.id.diff
Sat, Mar 8, 6:32 PM
F15334235: D15355.diff
Sat, Mar 8, 7:44 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10457: Improve Harbomaster/Drydock UI/UX before release
Commits
Restricted Diffusion Commit
rPfdca684814da: Slightly improve Buildable list in Harbormaster
Summary

Ref T10457. This makes diffs/revisions show the revision as the buildable title, and commits show the commit as the title.

Previously, the title was "Buildable X".

Also makes icons/colors/labels more consitent.

Test Plan

Screen Shot 2016-02-26 at 12.20.32 PM.png (257×554 px, 31 KB)

Diff Detail

Repository
rP Phabricator
Branch
dry4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10864
Build 13395: Run Core Tests
Build 13394: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 37027.Feb 26 2016, 8:23 PM
epriestley retitled this revision from to Slightly improve Buildable list in Harbormaster.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10457: Improve Harbomaster/Drydock UI/UX before release.
chad accepted this revision.Feb 26 2016, 8:38 PM
chad edited edge metadata.

itsawesome

This revision is now accepted and ready to land.Feb 26 2016, 8:38 PM
epriestley added a comment.Feb 26 2016, 8:43 PM

Haha, we need an itsacceptable.

Closed by commit rPfdca684814da: Slightly improve Buildable list in Harbormaster (authored by epriestley, committed by epriestley). · Explain WhyFeb 27 2016, 3:11 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
storage/
9 lines
4 lines
harbormaster/
interface/
13 lines
query/
30 lines
storage/
4 lines
repository/
storage/
4 lines

Diff 37027

View Options

src/applications/differential/storage/DifferentialDiff.php

Loading...
View Options

src/applications/differential/storage/DifferentialRevision.php

Loading...
View Options

src/applications/harbormaster/interface/HarbormasterBuildableInterface.php

Loading...
View Options

src/applications/harbormaster/query/HarbormasterBuildableSearchEngine.php

Loading...
View Options

src/applications/harbormaster/storage/HarbormasterBuildable.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryCommit.php

Loading...
Phabricator · Built by Phacility