search

jedfoster / SassMeister

The Sass playground
http://sassmeister.com
199 stars 25 forks source link

Remove double scrollbar on editors #167

Closed lukewatts closed 8 years ago

lukewatts commented 8 years ago

The property "overflow-y: scroll !important" in the CSS rule .touch .main_body .ace_scroller is causing two scrollbars to appear...one of which is completely pointless. Simply removing this line fixes the problem.

lukewatts commented 8 years ago

@jedfoster Have you had a chance to look over this? It would clean up the UI considerably I feel.

jedfoster commented 8 years ago

@lukewatts can you provide more info on this? Specifically, what browser/platform and mouse/trackpad?

lukewatts commented 8 years ago

I think it's because I have a touchscreen laptop. It's not like that on my desktop machine. Is there really a need for scroll bars to be visible for touchscreen devices? You can't even scroll them with a finger.

It's Windows8.1 All browsers (Firefox/Firefox Dev/Chrome/IE10)

jedfoster commented 8 years ago

@lukewatts would you please take a look at http://sassmeister-beta.herokuapp.com/ and tell me if it looks any better?

lukewatts commented 8 years ago

Perfect! No more scroll bar on my laptop.

New design looks awesome. Nice work!