search

ckrack / fbootstrapp

HTML, CSS, and JS toolkit for facebook apps
http://ckrack.github.com/fbootstrapp/
Apache License 2.0
1.61k stars 173 forks source link

overflow-y:scroll #6

Closed ptarjan closed 12 years ago

ptarjan commented 12 years ago

Why do you have a forced scrollbar on the element? For pages that fit vertically, there is no need to have an ugly scrollbar.

You can try removing my CSS stylesheet from here to see what I mean: http://apps.facebook.com/imdb_top_two_fifty/

ckrack commented 12 years ago

Thanks for the catch, that was left in there from Bootstrap. Of course, since FB Apps are based on iframes this is usually not intended behaviour.

I'm pushing a fix now.

On Tuesday, January 17, 2012, Paul Tarjan wrote:

Why do you have a forced scrollbar on the element? For pages that fit vertically, there is no need to have an ugly scrollbar.

You can try removing my CSS stylesheet from here to see what I mean: http://apps.facebook.com/imdb_top_two_fifty/

Reply to this email directly or view it on GitHub: https://github.com/ckrack/fbootstrapp/issues/6