search

Automattic / antiscroll

OS X Lion style cross-browser native scrolling on the web that gets out of the way.
1.07k stars 162 forks source link

Native scrollbar not being hidden in latest Chrome on Lion #46

Open davemckenna01 opened 11 years ago

davemckenna01 commented 11 years ago

Looks like the latest Chrome update has something funny going on with hiding native scrollbars. See screenshot - you get the custom antiscroll scrollbar overlaid on top of the native one. Screenshot is from Chrome 25.0.1364.99.

A co-worker has 24.0.1312.56 and this problem doesn't occur.

Related to issue #40 and the fix proposed in that commit, but the fix doesn't work on Lion.

antiscroll-chrome

Radagaisus commented 11 years ago

@davemckenna01 Issue #40 has a proposed commit that fixes this.

davemckenna01 commented 11 years ago

@Radagaisus @pirxpilot The problem still persists in Chrome 25 on Lion. I'm looking in to it, but so far no luck.

Radagaisus commented 11 years ago

@davemckenna01 Hi, we use a slightly modified version of Antiscroll, but I just copy pasted the suggested fix into our code and it worked.

davemckenna01 commented 11 years ago

@Radagaisus on Lion though? B/c I copy pasted the code as well and no change. But I did just get it to work with the CSS fix @epitrochoider suggested in the comments in issue #40 :

.box-wrap ::-webkit-scrollbar { width: 0px; height:0px; }

However, maybe the following would be a better solution to include in the library css file:

.antiscroll-wrap ::-webkit-scrollbar { width: 0px; height:0px; }

Frobitz commented 11 years ago

Does this actually remove the scrollbar though? I'm using the latest version, Chrome 25 and Lion. On the demo page, if you select all the text inside the scrollable region with the mouse, the region scrolls to the right and the native scroll bar becomes visible. Screen Shot 2013-02-27 at 11 39 27

davemckenna01 commented 11 years ago

My PR fixes this. The demo at http://learnboost.github.com/antiscroll/ continues to use a version of the code that is still broken in Chrome Lion.

Check out my branch chrome-25-lion-issues and open the demo index.html file, you won't see two scrollbars.

rauchg commented 11 years ago

Can we do a pull request for the demo as well? Thanks!

On Wed, Feb 27, 2013 at 2:06 PM, Dave McKenna notifications@github.comwrote:

My PR fixes this. The demo at http://learnboost.github.com/antiscroll/continues to use a version of the code that is still broken in Chrome Lion.

Check out my branch chrome-25-lion-issueshttps://github.com/davemckenna01/antiscroll/tree/chrome-25-lion-issuesand open the demo index.html file, you won't see two scrollbars.

— Reply to this email directly or view it on GitHubhttps://github.com/LearnBoost/antiscroll/issues/46#issuecomment-14203291 .

Guillermo Rauch LearnBoost CTO http://devthought.com

Frobitz commented 11 years ago

@davemckenna01 Ah, good to know. Thanks!

gnatih commented 11 years ago

@davemckenna01 I've updated the demo page to use the current antiscroll.js. Seems to fix the error on Windows/Chrome, can you confirm if it works for you (Lion/Chrome) ? We can then happily close this.

davemckenna01 commented 11 years ago

@gnatih the issue persists on Lion/Chrome 25, just checked. It's a weird bug - the only way I've found to fix it is in my PR. The solution seems crazy but it works.