Page MenuHomePhabricator

Remove 'Author:' byline text
ClosedPublic

Authored by chad on Jan 1 2015, 8:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 28 2025, 3:52 AM
Unknown Object (File)
Apr 26 2025, 1:53 AM
Unknown Object (File)
Apr 1 2025, 11:29 AM
Unknown Object (File)
Mar 30 2025, 7:10 AM
Unknown Object (File)
Mar 26 2025, 8:28 PM
Unknown Object (File)
Mar 18 2025, 1:31 PM
Unknown Object (File)
Feb 20 2025, 10:33 AM
Unknown Object (File)
Feb 15 2025, 12:10 AM

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rP61e26cd2421b: Remove 'Author:' byline text
Summary

The actual author here usually gets truncated by the extra text, which doesn't seem needed in most (all?) cases.

Test Plan

Look at list of recent commits.

Diff Detail

Repository
rP Phabricator
Branch
rem-author
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3462
Build 3469: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad retitled this revision from to Remove 'Author:' byline text.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 2 2015, 6:35 PM
This revision was automatically updated to reflect the committed changes.

Author name is mostly visible now, but e-mail (at least for Git repositories) is truncated as well.

Maybe we should display recognized Phabricator username (with a link) as we're doing for list of Differential Revisions? And when user isn't detected, then we show original commiter as now.

Right now with "Author: " text removed it doesn't look too consistent with Differential Revisions list.