Closed spacedino closed 6 years ago
I like that! Is this possible with either goToSlide() if we put it into amp-carousel, or a modification we can make to scrollTo() if we don't? (though since horizontal scrolling is broken inside the viewer until the layers work is released, I'm thinking we'd want to put it into amp-carousel regardless?)
WDYT, @camelburrito ?
I think just a scrollable carousel with hidden scroll bar and opaque arrows should do the trick , but the ends would always be opaque
On Fri, Nov 3, 2017 at 8:51 AM ericlindley-g notifications@github.com wrote:
I like that! Is this possible with either goToSlide() if we put it into amp-carousel, or a modification we can make to scrollTo() if we don't? (though since horizontal scrolling is broken inside the viewer until the layers work is released, I'm thinking we'd want to put it into amp-carousel regardless?)
WDYT, @camelburrito https://github.com/camelburrito ?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ampproject/ampstart/issues/740#issuecomment-341744161, or mute the thread https://github.com/notifications/unsubscribe-auth/AA_T_zIw-qppRGKwrc_vTKIRfNxQzFNUks5syzaPgaJpZM4QQaTq .
-- -- Sriram
increase translucent area so user has bigger tap target.
Planning to drop in an amp-carousel
and use the gradient as the "next" arrow.
Current If a user doesn't understand the scroll affordance and they click the farthest right edge, its a little weird to just select whats under it if we don't know if they intended to advance or view that page.
Should we? Make it so if the tap the edge we auto scroll the other filters? cc @ericlindley-g @mattludwig @camelburrito