search

tmcw / docbox

REST API documentation generator
ISC License
1.14k stars 191 forks source link

add overflow rule to contain #10

Closed Haroenv closed 8 years ago

Haroenv commented 8 years ago

right now scrolling the sidebar means hitting a 1px target. If the overflow is auto (and thus scrolling) you can scroll in there with your scroll wheel.

tmcw commented 8 years ago

Your scroll wheel doesn't work for the sidebar? Mine certainly does, as well as trackpad scrolling, in Chrome on a Mac.

Haroenv commented 8 years ago

Well in Safari, for example this page doesn't act as a regular "overflow inside an overflow" would. I did notice that the style changed since I made this pull request, I'm not sure if the fix would still do it. In Chrome I do get intended behaviour

tmcw commented 8 years ago

Thanks - implemented a similar fix in ffdf4a6123b443f296fd22f8c1bf57720ac979ce which removes the fancy scrollbars entirely.

Haroenv commented 8 years ago

I assume this needs some time to work through all the pages? The link I posted last time still has the old way of sidebar?

Anyways, thanks for making this easier