Hacker News Reader

Comment by rch | Hacker News Reader

It looks to me like pinch/zoom doesn't work. Not a big deal, but I wouldn't call it perfect until it's flawless on a tablet and phone.

View on HN

What do you mean pinch/zoom?

The page itself (not the slider), sets a fixed viewport, which means that a user cannot pinch/zoom the page on a tablet/phone.

Here is the code:

  <meta name="viewport" content="width=device-width, initial-scale=1, user-scalable=0">

Removing that little "user-scalable=0" should fix it, as being able to zoom in on a page is a nice feature, and shouldn't be disabled unless there is a really good reason.

Well that's not a plugin problem, thats a demo page problem. And I typically disable that when there is responsive/mobile formatting in place

I dunno, I want to see it work for real. It's whoever wrote the demos fault really. Throw some integration tests up and drive it through its paces. I bet you there's more missing areas.

You aren't getting this are you? The carousel doesn't have anything to do with pinch or zoom. If you put it on your site, boom , pinch and zoom your face off. I chose to disable pinch and zoom on MY demo website, because it is already formatted for mobile.

Makes sense - I should have looked more closely. I was checking it out on a tablet since that's where I'm most interested in these controls. I should mention that the swipe behavior of this one is nicer than most, and so I'll definitely be trying it out at some point. I might make a few modifications to better deal with phones, but that's about it.

Reply on news.ycombinator.com