Page MenuHomePhabricator

Homepage sprucing, spacing normalization
ClosedPublic

Authored by chad on Jan 21 2014, 9:51 PM.
Tags
None
Referenced Files
F14033399: D8034.id18157.diff
Sat, Nov 9, 5:35 PM
F14032899: D8034.id18166.diff
Sat, Nov 9, 4:08 PM
F14031682: D8034.id18167.diff
Sat, Nov 9, 11:43 AM
F14031659: D8034.id.diff
Sat, Nov 9, 11:40 AM
F14031351: D8034.diff
Sat, Nov 9, 10:07 AM
F14030987: D8034.diff
Sat, Nov 9, 8:52 AM
F14028405: D8034.diff
Fri, Nov 8, 1:08 PM
F14012108: D8034.diff
Fri, Nov 1, 7:19 AM
Tokens
"Haypence" token, awarded by btrahan.

Details

Summary

Cleans up the homepage a little bit. Removes the subheaders and buttons, links the panel header, and adds an icon for further hinting. Also aligned things up to the common 16px gutter.

Test Plan

Tested home, differential, and maniphest. Screenshotted changes

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Not sold on the icon, but tossing this up first for feedback.

I don't love the icon either but everything else seems fine to me.

I can leave it out and see if we get confused users.

(the header is still linked)

Yeah, works for me. It's not bad or anything, it just doesn't look quite like it belongs. Might just take a little getting used to. I think this is a step forward even if we take another step on the icon later.

That comment made little sense because I am bad at reading. I think either removing it or retaining it are fine. I'd maybe try removing it and wait for pushback, as you suggest.

chad updated this revision to Unknown Object (????).Jan 21 2014, 10:17 PM
  • Remove icon