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
F15405476: D15495.id37349.diff
Tue, Mar 18, 11:33 AM
F15391906: D15495.id.diff
Sat, Mar 15, 12:17 PM
F15385998: D15495.id37348.diff
Fri, Mar 14, 11:49 PM
F15380423: D15495.id37347.diff
Fri, Mar 14, 2:59 AM
Unknown Object (File)
Feb 22 2025, 1:00 PM
Unknown Object (File)
Feb 9 2025, 1:03 AM
Unknown Object (File)
Feb 9 2025, 1:03 AM
Unknown Object (File)
Feb 9 2025, 1:02 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 37349

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