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

Fix mobile issues #629

Closed
wants to merge 5 commits into from
Closed

Fix mobile issues #629

wants to merge 5 commits into from

Conversation

nemchik
Copy link

@nemchik nemchik commented Jul 3, 2017

I closed my other PRs because they were against the master branch and the commits were starting to add up.

These changes are all related to #611 and #189

To test and verify, open Hydra in Chrome, press Ctrl + Shift + J (opens the developer console) then press Ctrl + Shift + M (toggles mobile mode).

Most of these changes have little or no effect on desktop (larger resolution) viewers, but they will improve the UX for mobile users.

P.S. as of creating this PR, your develop branch is 2 commits behind master.

@theotherp
Copy link
Owner

You're awesome. Will merge in a couple of days.

theotherp pushed a commit that referenced this pull request Jul 8, 2017
@theotherp
Copy link
Owner

I have added the changes manually. Thanks.

@theotherp theotherp closed this Jul 8, 2017
@nemchik nemchik deleted the fixMobileIssues branch July 20, 2017 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants