Skip to content

✨ [Amp story] Scaffold desktop one panel experiment#34755

Merged
processprocess merged 11 commits intoampproject:mainfrom
processprocess:desktop-one-panel
Jun 10, 2021
Merged

✨ [Amp story] Scaffold desktop one panel experiment#34755
processprocess merged 11 commits intoampproject:mainfrom
processprocess:desktop-one-panel

Conversation

@processprocess
Copy link
Copy Markdown
Contributor

@processprocess processprocess commented Jun 8, 2021

Scaffolds files and tests for amp-story-desktop-one-panel experiment.

  • New CSS class on story element
  • New CSS file
  • Template with experiment toggle
  • Basic visual test
  • Basic unit test

@amp-owners-bot
Copy link
Copy Markdown

amp-owners-bot bot commented Jun 8, 2021

Hey @gmajoulet, @newmuis! These files were changed:

extensions/amp-story/1.0/amp-story-desktop-one-panel.css
extensions/amp-story/1.0/amp-story-store-service.js
extensions/amp-story/1.0/amp-story.css
extensions/amp-story/1.0/amp-story.js
extensions/amp-story/1.0/test/test-amp-story.js

@processprocess processprocess requested a review from jridgewell June 9, 2021 14:39
@processprocess
Copy link
Copy Markdown
Contributor Author

cc @jridgewell for OWNERS approval on config.js.

@processprocess processprocess requested review from danielrozenberg and removed request for jridgewell June 9, 2021 18:13
Copy link
Copy Markdown
Member

@danielrozenberg danielrozenberg left a comment

Choose a reason for hiding this comment

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

Approval for build-system/test-configs/config.js

@processprocess processprocess merged commit 5b263c3 into ampproject:main Jun 10, 2021
@processprocess processprocess deleted the desktop-one-panel branch June 10, 2021 12:24
westonruter added a commit to westonruter/amphtml that referenced this pull request Jun 11, 2021
…ebook-like-bento-version

* 'main' of github.com:ampproject/amphtml:
  minor updates + fix broken links (ampproject#34840)
  Add "wrapper": "bento" option to Bento components (ampproject#34838)
  ♻️ Move src/layout into core to unblock buildDOM for amp-layout (ampproject#34818)
  ♿ Apply `lang="en"` to relevant snippets in `test/` (ampproject#34768)
  Bento: Enable `npm` for `amp-video` (ampproject#34822)
  ✨[story-ads] Introduce new yellow segment progress bar v2 (ampproject#34804)
  SwG Release (ampproject#34825)
  📦 Update build-system devDependencies to v7.14.5 (ampproject#34802)
  Disable viewport warnings in experiment. (ampproject#34809)
  Apply lang="en" to examples/ (ampproject#34759)
  ✨ [Amp story] Scaffold desktop one panel experiment (ampproject#34755)
  ♻️ Migrate Style and DOM helpers into core/DOM + type-checking (ampproject#34681)
  🏗 Don't pull all externs into experiments (ampproject#34800)
  typechecking: remove pride as not compatible with rest of strategy (ampproject#34787)
  Fix forbidden terms to unblock `main` (ampproject#34799)
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