Page MenuHomePhabricator

Shrink side-nav on home when durable column is open
ClosedPublic

Authored by chad on Feb 27 2015, 3:31 AM.
Tags
None
Referenced Files
F14347838: D11901.diff
Thu, Dec 19, 3:38 AM
Unknown Object (File)
Wed, Dec 4, 7:36 PM
Unknown Object (File)
Sun, Dec 1, 3:41 PM
Unknown Object (File)
Sun, Dec 1, 3:00 PM
Unknown Object (File)
Sun, Dec 1, 3:41 AM
Unknown Object (File)
Wed, Nov 20, 11:29 PM
Unknown Object (File)
Nov 16 2024, 4:38 PM
Unknown Object (File)
Nov 9 2024, 11:53 PM
Subscribers

Details

Summary

This still needs some fine tuning, but wanted to get opinions. Using it on a laptop feels pretty good. This also moves durable-column.css into its own file since it'll likely continue to grow. Minor CSS tweaks to the near perfect rendition of durable column from pixel based mockups.

Test Plan

Press \ on my laptop. Having issues with Chrome however, but FF and Safari work as expected.

Screen_Shot_2015-02-26_at_7.28.48_PM.png (1×2 px, 431 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Shrink side-nav on home when durable column is open.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
chad added inline comments.
webroot/rsrc/css/application/base/standard-page-view.css
229

This doesn't trigger in Chrome, it's ... odd. If I toggle the rule on and off, it works. Assume to be a Chrome bug and need to find a work around.

  • split durable column into own css
btrahan edited edge metadata.

UI is fine with me.

It would be nice to land this so I can work on T7014 without having to e.g. merge in a CSS file re-factor. :(

This revision is now accepted and ready to land.Feb 27 2015, 7:08 PM

At first glance I don't like this very much, but we can muck with it later if I don't get used to it before we launch this.

What don't you like? The icon part on the left or?

I'd probably prefer an iconNav homepage outright (as an option for layout) non-dependent of Conpherence. The sidenav to me is fairly useless and I'd prefer the space on home.

Yeah -- the side nav is the most useful part of the homepage for me by a wide margin. I have plenty of space for the dashboard even on my laptop, so this feels like it's turning the useful part into an icon hunt and that I'm getting nothing in exchange.

chad edited edge metadata.
  • split durable column into own css
  • Move to ConpherenceDurableColumnView

I removed the icon magic and made this "Conpherence" since I sort of assume it completely goes away when you uninstall Conpherence.

Cool, I like the rename a bunch. :D

This revision was automatically updated to reflect the committed changes.