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

[Forwardport] Responsive Design Footers bottom of screen on mobile devices #15118 #16994

Conversation

ihor-sviziev
Copy link
Contributor

Original Pull Request

#15353
Solved footer issue for mobile view

Description

Solved footer issue for mobile view

Fixed Issues (if relevant)

  1. Responsive Design, Footers do not snap to bottom of screen on mobile devices #15118: Responsive Design, Footers do not snap to bottom of screen on mobile devices
  2. ...

Manual testing scenarios

  1. Check footer in mobile

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@magento-engcom-team
Copy link
Contributor

Hi @ihor-sviziev. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

  • @magento-engcom-team give me test instance - deploy test instance based on PR changes
  • @magento-engcom-team give me {$VERSION} instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

@gwharton
Copy link
Contributor

Arghhh, this resolution is not right. Please look at the original issue #15118. The PR that was originally created, that this is a forward port of, is a poor fix for this issue. It means that the footer is ALWAYS at the bottom of the screen, and the main page content scrolls behind it.

The resolution to this issue should in my opinion make the footers replicate the same behaviour as the desktop. See the remainder of the comments in #15118 and the second attempt at a PR which has not yet been accepted.

@ihor-sviziev
Copy link
Contributor Author

ihor-sviziev commented Jul 23, 2018

Hi @gwharton,
I created forwardport to 2.3-develop branch in order to have similar changes and same bugs in 2.2.x and 2.3.x and easier backport/forwardport fixes for them.
Here is two potential PR that fixes this issue: #16981 and #17006

@gwharton
Copy link
Contributor

Thanks @ihor-sviziev I see the need to keep the branches consistent.

It just feels a bit messy at the moment as there are several PR's that all seem to be conflicting with one another, and all in the "accept" state, and all proposing to make changes to 2.2-develop that will ultimately conflict with the changes being proposed to 2.3-develop here.

Aslong as someone somewhere has a good handle on all this, then I'm sure it will all workout in the end.!!! :)

@magento-engcom-team
Copy link
Contributor

Hi @ihor-sviziev. Thank you for your contribution.
We will aim to release these changes as part of 2.3.0.
Please check the release notes for final confirmation.

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

Successfully merging this pull request may close these issues.

5 participants