Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
davemckenna01 opened this issue Feb 25, 2013 · 10 comments
Open

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

davemckenna01 opened this issue Feb 25, 2013 · 10 comments

Comments

@davemckenna01
Copy link
Contributor

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
Copy link
Contributor

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

@davemckenna01
Copy link
Contributor Author

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

@Radagaisus
Copy link
Contributor

@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
Copy link
Contributor Author

@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
Copy link

Frobitz commented Feb 27, 2013

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
Copy link
Contributor Author

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
Copy link
Contributor

rauchg commented Feb 27, 2013

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//issues/46#issuecomment-14203291
.

Guillermo Rauch
LearnBoost CTO
http://devthought.com

@Frobitz
Copy link

Frobitz commented Feb 28, 2013

@davemckenna01 Ah, good to know. Thanks!

@gnatih
Copy link
Contributor

gnatih commented Apr 3, 2013

@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
Copy link
Contributor Author

@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.

rauchg added a commit that referenced this issue Apr 10, 2013
Give scrollbar pseudo-classes their own blocks, fixes issue #46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants