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

horizontal scrolling not working in safari 6? #9

Closed
onedrop opened this issue Aug 17, 2012 · 25 comments
Closed

horizontal scrolling not working in safari 6? #9

onedrop opened this issue Aug 17, 2012 · 25 comments
Assignees
Labels

Comments

@onedrop
Copy link

onedrop commented Aug 17, 2012

Having an issue in safari 6 on os x mountain lion. For example, when trying to issue a scrollTo() command in a 2000/2000px html body, only the vertical component is scrolled. Is anyone else experiencing this? It works find in safari 5, ie9, firefox and chrome.

$.scrollTo({left:1000, top:1000}, 1000);

@flesler
Copy link
Owner

flesler commented Aug 18, 2012

Haven't heard of this, I'll try to test this on Safari 6 asap (gotta get it first) got a link to your site?

@Moinax
Copy link

Moinax commented Sep 13, 2012

I confirm that there is a bug with Safari Version 6.0 (8536.25)

It seems to work on Safari Version 6.0 (7536.25)

I think we should blame Apple on this one

@flesler
Copy link
Owner

flesler commented Sep 13, 2012

Yeah, still if they don't fix that in further versions, we'll need a hack
anyways.
If someone figures a workaround, let me know, I'll try to test this asap.

On Thu, Sep 13, 2012 at 7:50 AM, Jérôme Poskin notifications@github.comwrote:

I confirm that there is a bug with Safari Version 6.0 (8536.25)

It seems to work on Safari Version 6.0 (7536.25)

I think we should blame Apple on this one


Reply to this email directly or view it on GitHubhttps://github.com//issues/9#issuecomment-8524529.

Ariel Flesler

@kontur
Copy link

kontur commented Sep 18, 2012

Reporting the same problem, but I don't have Mountain Lion and thus Safari 6 myself, so hard to test what's going on. The browser also fails to scroll to the correct vertical position, when the call is made via a selector (i.e. $.scrollTo($('#some'))), if that element is off screen.

@flesler
Copy link
Owner

flesler commented Sep 18, 2012

@johannesneumeier got a site online to test this?

@kontur
Copy link

kontur commented Sep 18, 2012

Only a client project with a safari 6 work around in place. If I find the time I strip the content and upload it for reviewing, but can't promise I get around to it. Basically a page with 500% width and floated 100% width divs, stacked in several rows. Scrolling to any but the first element in each row would fail to scroll horizontally, but finds the right height ok.

@kontur
Copy link

kontur commented Oct 1, 2012

The following example page scrolls sideways to the 5th box of the top row on load.
In Safari version 8536.25 the page scrolls down a little (vertical scrolling ok), but fails to scroll to the right (horizontal scrolling fail).

http://serveri.magis.com/johannes/scrollto-safari6-bug/
(Edit: Sorry, link no longer available)

Hope this helps to pin down what's the problem. Version 1.4.3.1 of scrollTo and 1.8.1 of jQuery used.

@zehfernandes
Copy link

nothing about this bug?

@flesler
Copy link
Owner

flesler commented Oct 10, 2012

Works for me on 6.0 (7536.25) Mac. I'll try updating it right now.

@zehfernandes
Copy link

I'm test on safari 6.0 Ipad.
Works when used just vertical scrolling when add horizontal broken

If helps the demo: http://zehfernandes.com/jquery.fullContent/#home

@hilko
Copy link

hilko commented Oct 11, 2012

I'm having the exact same issue, but with Google Chrome Canary (so possibly a near-future Chrome issue!)

@Franz-Kafka
Copy link

I have exactly the same problem. Is there a fix for that?

@hongaar
Copy link

hongaar commented Nov 8, 2012

Posted a workaround on Stack Overflow, but I don't consider this well-tested enough to prepare a pull request. The post is here: http://stackoverflow.com/questions/13296144/set-scrollleft-scrolltop-simultaneously-does-not-work-in-safari-6, a patched jquery.scrollTo.js here: http://nabble.nl/demo/safari6scrollto/scrollto.js.

@flesler
Copy link
Owner

flesler commented Nov 8, 2012

Ok thanks, as you said, that surely needs polishing. I'll keep this in my TODO list, will let you know if I can get this in.

@959designs
Copy link

Not working in Safari Version 6.0.1 (8536.26.14) or Mobile Safari

@spilliton
Copy link

Also having this issue. The patched one from @hongaar appears to fix it.

@959designs
Copy link

Yeah I used his fix as well and it solved all of my problems. His fix could use some work though so maybe if I get some time I can work on it as well, but honestly at this point there really is no reason for it not to warrant a pull. It definitely works

@klihelp
Copy link

klihelp commented May 23, 2013

Any update on this?

@flesler
Copy link
Owner

flesler commented May 23, 2013

I tried with my version of Safari and worked, don't quite have the time to rollback my Safari, reproduce and roll a hack and re-test it on the rest too see none broke. If you can contribute code it will speed up the process.

@ghost
Copy link

ghost commented Oct 5, 2013

Any progress?

@Franz-Kafka
Copy link

any news?

@ghost ghost assigned flesler Jan 13, 2014
@flesler
Copy link
Owner

flesler commented Mar 4, 2015

Hi,

On #101 we're testing a new beta version (currently 1.5.2-beta) that fixes scrolling in Chrome 40 (was working on most ends but not all), also works correctly on iOS and Android which wasn't previously working. I tested this version with my Safari 8.0 and it works.

I'd like you to join us on that other issue and run the tests on your available browsers (including Safari 6), I won't be releasing this until it gets enough confirmation from users.

Thank you

@klihelp
Copy link

klihelp commented Mar 5, 2015

The new version looks good

@flesler
Copy link
Owner

flesler commented Mar 5, 2015

You tested it on Safari 6?

@flesler
Copy link
Owner

flesler commented Mar 18, 2015

I merged 2.0.0 to master via #109 and this issue might get fixed (indirectly).
If the problem persists please reopen the issue with details on the situation.
If you have any issue after the update, first check this link, If your problem is not solved then go ahead and report the issue.

@flesler flesler closed this as completed Mar 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests