search

chutterbox / chutter

https://chutter.co
MIT License
12 stars 4 forks source link

changed side nav, fixed double scroll bars #18

Closed mentos1386 closed 9 years ago

mentos1386 commented 9 years ago

Fix: main app

Fixed visual bug in sidenav, it had double scroll bars. Added overflow:hidden to remove one. (seen in firefox) Bug image

EDIT: overflow:hidden was responsible for hiding first item from list (science) and had to add margin-top: 64px (height of navbar) to fix it.

chutterbox commented 9 years ago

@mentos1386 thanks for pointing this out. I see what you're talking about. There seems to be two issues:

1) the first two(ish) networks are being cut off in firefox

2) double scroll bars

i think you're onto the right approach, but the toolbar shifts if there is scrollable content in the middle section, and i use transform: translatey to handle that interaction (see nav-controllers).

the most immediate issue is that the #scrolly-left and #scrolly-right ids don't have the -moz-transform property in chutter.sass. (it has -webkit-transform and transform, but is missing -moz-transform), adding this back in would fix number 1)

the second issue may mean we need a little bit better solution, ideally this is the toolbar behavior we should be seeing: https://www.youtube.com/watch?v=eR9K1UKTe0c

chutterbox commented 9 years ago

i think this is resovled/rendered obsolete by the new layout and what not. will reopen if we still need it