Skip to content

♻️ Alias video.fullscreen action for symmetry with interface names#27017

Merged
alanorozco merged 5 commits intoampproject:masterfrom
alanorozco:fullscreenenter
Feb 27, 2020
Merged

♻️ Alias video.fullscreen action for symmetry with interface names#27017
alanorozco merged 5 commits intoampproject:masterfrom
alanorozco:fullscreenenter

Conversation

@alanorozco
Copy link
Copy Markdown
Member

Maintains external sanity, e.g. we can say "to implement any playback actions on amp-video-iframe implement the method with its corresponding name".

@alanorozco alanorozco merged commit 06f411a into ampproject:master Feb 27, 2020
@alanorozco alanorozco deleted the fullscreenenter branch February 28, 2020 17:09
robinvanopstal added a commit to jungvonmatt/amphtml that referenced this pull request Mar 2, 2020
* master:
  Launch `amp-next-page` v2 & clean up experiment (ampproject#27035)
  ✨Implement Display Locking on amp-accordion (ampproject#25867)
  📖<amp-next-page> documentation and validation (ampproject#26841)
  Improve visibility event tests (ampproject#26847)
  amp-geo: Fall back to API for country (ampproject#26407)
  ✨ Add customization options to <amp-story-quiz> (ampproject#26714)
  navigation: Minor test improvements (ampproject#26926)
  ♻️ Alias video.fullscreen action for symmetry with interface names (ampproject#27017)
  ✨ Implements `amp-analytics` support for `amp-next-page` (ampproject#26451)
  ✨ amp-video-iframe integration: global jwplayer() singleton by default (ampproject#26969)
  Fix unit tests broken by ampproject#26687 (ampproject#27000)
  Filter redirect info from emails (ampproject#27012)
  📖 Make amp-bind doc valid, fix amp-form stories filter (ampproject#27003)
  url-replacements: Minor test improvement (ampproject#26930)
  viewport: Minor test improvement (ampproject#26931)
  amp-consent fullscreen warning (ampproject#26467)
  dep-check: USE CAPS FOR IMPORTANCE (ampproject#26993)
  fix img url (ampproject#26987)

# Conflicts:
#	extensions/amp-next-page/amp-next-page.md
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.

4 participants