Page MenuHomePhabricator
Differential D15495

Make dates/times more concise in Diffusion
ClosedPublic
Actions

Authored by epriestley on Mar 18 2016, 4:00 PM.
Tags
None
Referenced Files
F19504385: D15495.id37349.diff
Fri, Jan 9, 5:06 AM
F19503527: D15495.diff
Fri, Jan 9, 12:17 AM
F19117845: D15495.id37348.diff
Dec 9 2025, 8:07 PM
F19113563: D15495.id37348.diff
Dec 9 2025, 6:13 PM
F19039052: D15495.diff
Nov 26 2025, 6:40 AM
F19025613: D15495.diff
Nov 24 2025, 8:12 AM
F19001580: D15495.diff
Nov 21 2025, 3:44 AM
F18851310: D15495.id.diff
Oct 30 2025, 11:45 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Commits
Restricted Diffusion Commit
rPf07d0ae7c3ae: Make dates/times more concise in Diffusion
Summary

I think I like this better -- but maybe right-aligned?

Test Plan

Screen Shot 2016-03-18 at 8.58.48 AM.png (1×2 px, 398 KB)

Screen Shot 2016-03-18 at 8.59.03 AM.png (1×2 px, 324 KB)

Diff Detail

Repository
rP Phabricator
Branch
shorttime
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11208
Build 13913: Run Core Tests
Build 13912: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 37347.Mar 18 2016, 4:00 PM
epriestley retitled this revision from to Make dates/times more concise in Diffusion.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
Herald edited edge metadata. · View Herald TranscriptMar 18 2016, 4:00 PM
chad accepted this revision.Mar 18 2016, 4:02 PM
chad edited edge metadata.

right aligned would be great

This revision is now accepted and ready to land.Mar 18 2016, 4:02 PM
epriestley updated this revision to Diff 37348.Mar 18 2016, 4:10 PM
epriestley edited edge metadata.
  • Right-align everything.

I could also maybe using a grey text for this or something? I think it's an improvement but it still doesn't look quite right to me. Maybe I just need to get used to it, though.

Closed by commit rPf07d0ae7c3ae: Make dates/times more concise in Diffusion (authored by epriestley, committed by epriestley). · Explain WhyMar 18 2016, 4:11 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 37348

View Options

src/applications/diffusion/controller/DiffusionLastModifiedController.php

Loading...
View Options

src/applications/diffusion/view/DiffusionBranchTableView.php

Loading...
View Options

src/applications/diffusion/view/DiffusionBrowseTableView.php

Loading...
View Options

src/applications/diffusion/view/DiffusionHistoryTableView.php

Loading...
View Options

src/applications/diffusion/view/DiffusionPushLogListView.php

Loading...
View Options

src/applications/diffusion/view/DiffusionTagListView.php

Loading...
View Options

src/applications/people/storage/PhabricatorUser.php

Loading...
Phabricator · Built by Phacility