-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
🐛 [Amp story] Null check on screen.orientation #35639
Merged
processprocess
merged 1 commit into
ampproject:main
from
processprocess:orientation-lock
Aug 12, 2021
Merged
🐛 [Amp story] Null check on screen.orientation #35639
processprocess
merged 1 commit into
ampproject:main
from
processprocess:orientation-lock
Aug 12, 2021
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
processprocess
added this to In progress
in wg-stories Desktop One Panel
via automation
Aug 12, 2021
Hey @gmajoulet, @newmuis! These files were changed:
|
processprocess
changed the title
🐛 [Amp story] Null check on screen.orientation.lock
🐛 [Amp story] Null check on screen.orientation
Aug 12, 2021
processprocess
requested review from
mszylkowski
and removed request for
gmajoulet and
newmuis
August 12, 2021 15:25
newmuis
approved these changes
Aug 12, 2021
mszylkowski
approved these changes
Aug 12, 2021
If this totally breaks the experience on impacted devices, let's file a P0 and request a cherry-pick. |
calebcordry
pushed a commit
that referenced
this pull request
Aug 12, 2021
(cherry picked from commit 4ed4b92)
calebcordry
pushed a commit
that referenced
this pull request
Aug 12, 2021
(cherry picked from commit 4ed4b92)
zoharnyego
added a commit
to apester-dev/amphtml
that referenced
this pull request
Aug 12, 2021
commit b9e41a2 Author: Ryan Cebulko <ryan@cebulko.com> Date: Thu Aug 12 16:47:01 2021 -0400 ♻️ Define helper for hash params and drop barely-used `getMode().log` (ampproject#35628) * Add getHashParams helper * Add win param * Update mode.js to use helper * Add tests * Update callsites * Remove getMode().log uses * Remove log from ModeDef type * Lint fixes * Fix tests * Split up isModeDevelopment commit 0e66e28 Author: Alan Orozco <alanorozco@users.noreply.github.com> Date: Thu Aug 12 13:25:34 2021 -0700 🚮 Remove `<amp-story>` check for `<amp-audio>` actions (ampproject#35643) Removes check from `<amp-audio>` that prevents actions when descending from an `<amp-story>` element. `<amp-story>` now has an action allowlist, so the check was redundant. The action is still prevented, triggering the message '[Action] "AMP-AUDIO.play" is not allowlisted []'. commit 6b46842 Author: honeybadgerdontcare <honeybadgerdontcare@users.noreply.github.com> Date: Thu Aug 12 13:19:24 2021 -0700 Update link to modifications best practices (ampproject#35647) * Update link to modifications best practices * Update dead links commit a59d776 Author: Raghu Simha <rsimha@amp.dev> Date: Thu Aug 12 14:31:55 2021 -0400 🏗 Move Edge testing from GH Actions to CircleCI (ampproject#35630) commit b6476c3 Author: Khoi Doan <khoid@google.com> Date: Thu Aug 12 13:47:16 2021 -0400 ✨ Add bitness to User Agent Client Hint Params in ad requests (ampproject#35612) * Add User Agent Client Hint Params to Google Ads. * Update the param names. * Add timeout for UACH promise (1 second). * Use fake timers for timeout test. * Lint * Add exception for dependency on promise.js. * Add bitness field to User Agent Client Hints params * Fix merge * Fix merge again * Lint commit 87d5d14 Author: Ryan Cebulko <ryan@cebulko.com> Date: Thu Aug 12 13:43:51 2021 -0400 ♻️ Remove `version` from legacy mode object (ampproject#35624) * Remove `version` from legacy mode object * Restore version to mode-object for now * Remove version entirely * Lint fixes * Fix transformer test fixtures * Fix test commit 00b8e7e Author: Brandon Suen <brandonbsuen@google.com> Date: Thu Aug 12 10:31:56 2021 -0700 📖 [Story interactive] Add Documentation for Image Quizzes and Polls (ampproject#35618) * Documentation text added * Add images and source code * Reorganize in alphabetical order * Reordering * Add additional attributes * Fix typo * Fix typo * Fix typo * Update screenshots * Add note about image sizes * Revise note about image size commit 9813cea Author: Ryan Cebulko <ryan@cebulko.com> Date: Thu Aug 12 13:27:55 2021 -0400 ♻️ #core form, modal, amp-element-helpers ("real" diff in desc) (ampproject#35608) * Update src/form code * Update src/modal code * Update src/video-interface code * Move src/{form,modal,amp-element-helpers} -> #core/dom * Fix types * update dep-check config * Remove bad assert * Update imports of form,modal,amp-element-helpers * Fix rebase error commit 4ed4b92 Author: Philip Bell <philip.hunter.bell@gmail.com> Date: Thu Aug 12 12:38:05 2021 -0400 Null check on orientation lock. (ampproject#35639) commit 5ff32ad Author: Chris Antaki <ChrisAntaki@gmail.com> Date: Wed Aug 11 15:12:58 2021 -0700 SwG Release 0.1.22.179 (ampproject#35629) commit 2106624 Author: Matias Szylkowski <mszylkowski@google.com> Date: Wed Aug 11 17:13:54 2021 -0400 📖 [Story sidebar] Removed sidebar from docs (ampproject#35616) * removed sidebar and bookend docs * removed unused import * revert test removals * Add newline on visual test html commit d21ecc0 Author: Ryan Cebulko <ryan@cebulko.com> Date: Wed Aug 11 17:01:02 2021 -0400 ♻️ #core: error helpers (ampproject#35621) * Define core/error helpers * Drop createErrorVargs * Restore name assignment in Log#error * Typechecking/extern * Remove unused error_ helper * Typo * Add @ returns commit 6404407 Author: Brandon Suen <brandonbsuen@google.com> Date: Wed Aug 11 13:33:02 2021 -0700 ✅ [Story interactive] Rename Example Image Quiz and Poll Stories to Use .amp.html Extension (ampproject#35511) * Rename examples to use .amp.html extension * Remove confetti from results
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
On iPads,
orientation
is not defined.An error is being thrown when trying to access the
screen.orientation.lock
property.This PR checks if
orientation
exists before accessinglock
.Related to the work in #35478