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
F14359430: D11901.id28672.diff
Fri, Dec 20, 9:24 AM
Unknown Object (File)
Thu, Dec 19, 7:17 AM
Unknown Object (File)
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)
Nov 20 2024, 11:29 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
Branch
duribles
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4666
Build 4680: [Placeholder Plan] Wait for 30 Seconds

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
234

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.