Fixes a few issues. The author of the commit is more prominent / not cut off. Auditors is in a more consistent location. More space is available for reasons. Commits by themselves look much less janky. Only downside is actual Audits are now 3 lines vs. 2, but the extra space is used well.
Details
Details
- Reviewers
epriestley btrahan - Commits
- Restricted Diffusion Commit
rPe2fcc3c1879c: Touch up Audit/Commit List UI
Test list of audits and commits.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
Is there any point in showing commit author name, used in VCS system (name & e-mail in git) VS resolved Phabricator username with a link? Then we could have kept 2 line element.
Here is how it looks (picture from my fork):
Comment Actions
I don't know the history of the code well enough to give you an answer. I would like to split "Commits" vs. "Commits with Audits" UI out further, but I don't know when we'd approach that. T6630 would likely be the place to pursue more improvements there.