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

Position:relative on body #5665

Merged
merged 2 commits into from Oct 18, 2016
Merged

Position:relative on body #5665

merged 2 commits into from Oct 18, 2016

Conversation

dvoytenko
Copy link
Contributor

@dvoytenko dvoytenko commented Oct 18, 2016

Closes #5667.

This PR adds body{position:relative} CSS to ensure that content with position:absolute is positioned relative to the body and offset to be shown under the viewer's header, not behind it.

Copy link
Member

@cramforce cramforce left a comment

Choose a reason for hiding this comment

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

LGTM
but update PR description and reference bug if possible.

@dvoytenko dvoytenko merged commit 7481c30 into ampproject:master Oct 18, 2016
@dvoytenko dvoytenko deleted the viewport12 branch October 18, 2016 21:30
samiamorwas pushed a commit to samiamorwas/amphtml that referenced this pull request Oct 18, 2016
…mp_reddit_extension

* 'master' of https://github.com/ampproject/amphtml: (63 commits)
  Position:relative on body (ampproject#5665)
  Fix for ampproject#5663 (ampproject#5664)
  Nokta Ad Server is added as an ad type (ampproject#5550)
  RFC: Separate the load phase of AMP into multiple chunks. (ampproject#5536)
  Add OWNERS files for A4A. (ampproject#5651)
  Update DEVELOPING.md
  Call original event add/remove via interceptor (ampproject#5650)
  Fix wording on confusing steps to protect against CSRF. (ampproject#5646)
  Install runtime CSS for all amp tests (ampproject#5642)
  Implement outgoing link URL replacements. (ampproject#5628)
  Wait for window to load before installing ServiceWorker. (ampproject#5638)
  iOS wrapper viewport implementation (ampproject#5629)
  Do not use AMP Version as RTV Versions (ampproject#5474)
  Purch Ad Support for Amp-Ads (ampproject#5464)
  A11Y fix for sticky ad close button. (ampproject#5640)
  Propagate ARIA attributes to real-elements (ampproject#5590)
  Ibillboard integration (ampproject#5392)
  Add some keywords to the NPM description of the validator. (ampproject#5633)
  PWA: messaging and broadcast (ampproject#5588)
  Carousel swipe not working well on Android Firefox (ampproject#5626)
  ...
Lith pushed a commit to Lith/amphtml that referenced this pull request Dec 22, 2016
* Position:relative on body

* reference bug
Lith pushed a commit to Lith/amphtml that referenced this pull request Dec 22, 2016
* Position:relative on body

* reference bug
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.

None yet

2 participants