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

Hamburger menu responsiveness #628

Closed
clash99 opened this issue Sep 1, 2016 · 4 comments
Closed

Hamburger menu responsiveness #628

clash99 opened this issue Sep 1, 2016 · 4 comments
Assignees
Milestone

Comments

@clash99
Copy link
Contributor

clash99 commented Sep 1, 2016

Steps to reproduce the error

View platform on iphone.

Actual behavior

See hamburger menu not in header area on iphone (viewed on iphone6). Looks alright in replicated responsive views but for some reason, not the same on actual phone :(

Expected behavior

Adjust css so hamburger menu stays in header. (After user in signed in, behavior is as expected)

Actual:
canvas
Emulated in desktop web browser:
screenshot 2016-09-01 09 56 15

@clash99
Copy link
Contributor Author

clash99 commented Sep 1, 2016

While in the area, ideas to improve usability:

  • Add border around hamburger
  • Reduce project name size

@clash99 clash99 self-assigned this Sep 1, 2016
@clash99 clash99 modified the milestones: Sprint 9, Sprint 8 Sep 1, 2016
@clash99
Copy link
Contributor Author

clash99 commented Sep 7, 2016

Appears to be a safari error only.

ian-ross added a commit that referenced this issue Sep 8, 2016
Safari fix for header menu in small screens
@seav
Copy link
Contributor

seav commented Sep 8, 2016

Is this issue resolved by PR #651? (Note the PR follow-up action: "Once released to staging, qa in mobile safari browser")

@clash99
Copy link
Contributor Author

clash99 commented Sep 9, 2016

Passed QA in mobile Safari browser on staging. Closing this task.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants