Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

iOS: mechanical md-sidenav (menu) scrolling #7786

Closed
Hammild opened this issue Mar 29, 2016 · 1 comment
Closed

iOS: mechanical md-sidenav (menu) scrolling #7786

Hammild opened this issue Mar 29, 2016 · 1 comment
Assignees
@crisbeto

Comments

@Hammild
Copy link

Hammild commented Mar 29, 2016

Scrolling in the md-sidenav menu in iOS is very mechanical compared to Android. On an Android device, a scroll will decelerate before it comes to a halt. On a iOS device, a scroll stops abruptly which is in conflict with the guidelines (MD-Mass and weight).

How to reproduce:

  1. Open the md-sidenav menu (not the sidenav component)
  2. Open the "Demo"-accordian to make the md-sidenav scrollable
  3. Scroll up and down
  4. Observe that the scroll stops instantly when using a iOS device

Tested on:
Version: Angular Material 1.1.0-RC1
Model: iPad mini
Browser: Safari 9.3
Operating system: 9.3 (13E237)

Tested on:
Version: Angular Material 1.1.0-RC1
Model: iPhone 6
Browser: Safari 9.3
Operating system: 9.3 (13E233)
@crisbeto
Copy link
Member

Wasn't aware that we had this on sidenav as well, I added #7751 recently to address the same issue on the demos. I'll take care of this one.

@crisbeto crisbeto self-assigned this Mar 29, 2016
@crisbeto crisbeto added the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Mar 29, 2016
crisbeto added a commit to crisbeto/material that referenced this issue Mar 29, 2016
@crisbeto
fix(sidenav): enable native scrolling on touch devices
d9d4ff7 
Enables the native inertia scrolling in the sidenav.

Closes angular#7786.
@crisbeto crisbeto mentioned this issue Mar 29, 2016
Closed
ThomasBurleson pushed a commit that referenced this issue Apr 1, 2016
@crisbeto @ThomasBurleson
fix(sidenav): enable native scrolling on touch devices
573bead 
Enables the native inertia scrolling in the sidenav.

Closes #7786.

Closes #7751
ThomasBurleson pushed a commit that referenced this issue Apr 1, 2016
@crisbeto @ThomasBurleson
fix(sidenav): enable native scrolling on touch devices
e440edb 
Enables the native inertia scrolling in the sidenav.

Closes #7786.

Closes #7751
gmoothart pushed a commit to gmoothart/material that referenced this issue Apr 5, 2016
@crisbeto @gmoothart
fix(sidenav): enable native scrolling on touch devices
4d9c0f0 
Enables the native inertia scrolling in the sidenav.

Closes angular#7786.

Closes angular#7751
@ThomasBurleson ThomasBurleson removed the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Apr 25, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@crisbeto crisbeto

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ThomasBurleson @crisbeto @Hammild