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

[RFC] Initial papercut fixes for mobile #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[RFC] Initial papercut fixes for mobile #2

wants to merge 3 commits into from

Conversation

maxcrees
Copy link
Contributor

@maxcrees maxcrees commented Dec 22, 2016

NB: A live preview of these changes is available at g7sweb.uwtb.net

Any additional recommended changes are requested.

Description (from commit message)

Changes for mobile:

  • The header links are now visible
  • The footer is anchored to the bottom of the page with smaller
    font-size
  • Some margins were added to various boxes that end up stacking on
    mobile

Changes for desktop:

  • The issue where text was being cut-off should be fixed (for real this
    time!)

Global changes:

  • The copyright notice was moved to the right-side of the footer in
    anticipation of links being added to the left side so that the links
    will appear before the copyright notice on mobile
  • Update template with the above change and add mobile.css

Changes for mobile:
* The header links are now visible
* The footer is anchored to the bottom of the page with smaller
  font-size
* Some margins were added to various boxes that end of stacking on
  mobile

Changes for desktop:
* The issue where text was being cut-off should be fixed (for real this
  time!)

Global changes:
* The copyright notice was moved to the right-side of the footer in
  anticipation of links being added to the left side so that the links
  will appear before the copyright notice on mobile
* Update template with the above change and add mobile.css
@maxcrees
Copy link
Contributor Author

maxcrees commented Dec 22, 2016

A fix to properly place the footer at the bottom of the screen when content does not fill the viewport (e.g. here) is in the works. Please do not merge until this commit is made.

@maxcrees
Copy link
Contributor Author

Commit sroracle/galapagos-linux.github.io@35ba07a4883b91ae8af50ea8b3a3741241865d65 fixes the above issue.

Copy link
Contributor

@awilfox awilfox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great; I really appreciate it! I've found two small issues:

  • The footer seems to have no padding now, and the text runs against the side of the screen.
  • Portrait-oriented tablets don't always work.

@@ -0,0 +1,56 @@
@media only screen and (max-device-width: 750px) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why 750px?

This seems to work okay on mobiles in both orientation, but on some tablets (Nook, Fire, iPad/iPad mini), it looks weird in portrait mode because the tablet is 768 screen pixels. Namely, the hero columns are close together (they need the margin-bottom at line 14).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 750px limit is arbitrary, it's just to detect if the device is mobile or not. Would bumping it to 800px solve the tablet problem?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

768px is probably good. That is the width of older laptops/netbooks and most tablets, which would all benefit from this layout.

bottom: 0;
height: 5em;
font-size: 10pt;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we have a small amount of padding here? Not sure why, but it seems to have none - at least on Fennec.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Totally agree on this. Checkout the demo page now and see if that's better (clearing cache if necessary).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to have broken on 320x480: Screen shot 2016-12-31 at 00 44 48.png
:(

* mobile stylesheet max-device-width changed from 750px to 768px to
  account for some tablets
* footer: add horizontal padding and fix anchoring on 320x480 resolution
* big link buttons: fix for some weird sizing on some resolutions
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