Page MenuHomePhabricator
Differential D13382

Smooth out some UI/UX issues in Harbormaster
ClosedPublic
Actions

Authored by epriestley on Jun 21 2015, 8:47 PM.
Tags
None
Referenced Files
F15516648: D13382.id.diff
Fri, Apr 18, 8:18 PM
F15512085: D13382.diff
Thu, Apr 17, 7:59 AM
F15481660: D13382.id32397.diff
Tue, Apr 8, 10:21 PM
F15473279: D13382.id32460.diff
Sat, Apr 5, 8:51 PM
F15461192: D13382.id32397.diff
Tue, Apr 1, 6:03 AM
F15445321: D13382.diff
Thu, Mar 27, 1:26 PM
F15412259: D13382.diff
Mar 19 2025, 12:04 PM
F15407139: D13382.id.diff
Mar 18 2025, 4:00 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T8096: Improve Harbormaster UI/UX (v1)
Commits
Restricted Diffusion Commit
rPc31e25d5ceed: Smooth out some UI/UX issues in Harbormaster
Summary

Ref T8096. Fixes a few bugs and glitches.

  • Set build completion time when handling a message.
  • Format duration information in a more human-readable way.
  • Use a table for build variables.
  • Fix up container PHIDs on diffs (a touch hacky, should be OK for now though).
Test Plan

Browsed around the UI.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 32397.Jun 21 2015, 8:47 PM
epriestley retitled this revision from to Smooth out some UI/UX issues in Harbormaster.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T8096: Improve Harbormaster UI/UX (v1).
Herald added a subscriber: epriestley. · View Herald TranscriptJun 21 2015, 8:47 PM
btrahan accepted this revision.Jun 22 2015, 5:16 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 22 2015, 5:16 PM
Closed by commit rPc31e25d5ceed: Smooth out some UI/UX issues in Harbormaster (authored by epriestley, committed by epriestley). · Explain WhyJun 23 2015, 5:24 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
editor/
15 lines
harbormaster/
controller/
102 lines
28 lines
10 lines
engine/
5 lines
worker/
6 lines

Diff 32460

View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
View Options

src/applications/harbormaster/controller/HarbormasterBuildViewController.php

Loading...
View Options

src/applications/harbormaster/controller/HarbormasterBuildableViewController.php

Loading...
View Options

src/applications/harbormaster/controller/HarbormasterController.php

Loading...
View Options

src/applications/harbormaster/engine/HarbormasterBuildEngine.php

Loading...
View Options

src/applications/harbormaster/worker/HarbormasterTargetWorker.php

Loading...
Phabricator · Built by Phacility