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

side nav and app bar issues on iOS #4825

Closed
indirectlylit opened this issue Jan 29, 2019 · 6 comments
Closed

side nav and app bar issues on iOS #4825

indirectlylit opened this issue Jan 29, 2019 · 6 comments
Assignees
Labels
OS: iOS P1 - important Priority: High impact on UX
Milestone

Comments

@indirectlylit
Copy link
Contributor

indirectlylit commented Jan 29, 2019

Observed behavior

image

Expected behavior

  • side nav should be accessible and app bar

User-facing consequences

cannot use app

Steps to reproduce

load in iOS, click and scroll

This can be reproduced in browserstack on iOS, but only using http://kolibribeta.learningequality.org (not localhost - see #4826)

Context

0.12

@indirectlylit indirectlylit added P1 - important Priority: High impact on UX OS: iOS labels Jan 29, 2019
@indirectlylit indirectlylit added this to the 0.12.0 milestone Jan 29, 2019
@jonboiser
Copy link
Contributor

jonboiser commented Jan 29, 2019

The bar moves to the left when the scrolling changes position from fixed to absolute during the scroll. This probably happens because the page is usually bigger than the viewport.

On the profile page, where the main area isn't wider than the viewport, I don't see it move.

@jonboiser
Copy link
Contributor

As for the hamburger button not working, I think it's the 'scrolling-pane' class interfering with the Side Nav appearing. In browserstack, if you remove the 'scrolling-pane' class from the main div, side-nav appears when you click the hamburger.

@jonboiser
Copy link
Contributor

On Browserstack Chrome, clicking the search magnifying glass button doesn't work either.

@jonboiser
Copy link
Contributor

Removing the -webkit-overflow-scrolling: touch; // iOS momentum scrolling rule from .scrolling-pane seems to fix it.

@indirectlylit
Copy link
Contributor Author

okay, let's do that then and re-open #2279

@indirectlylit
Copy link
Contributor Author

fixed when #5100 is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OS: iOS P1 - important Priority: High impact on UX
Projects
None yet
Development

No branches or pull requests

2 participants