Skip to content

✨[story-ads] Introduce new yellow segment progress bar v2#34804

Merged
calebcordry merged 5 commits intoampproject:mainfrom
calebcordry:yellow-v2
Jun 10, 2021
Merged

✨[story-ads] Introduce new yellow segment progress bar v2#34804
calebcordry merged 5 commits intoampproject:mainfrom
calebcordry:yellow-v2

Conversation

@calebcordry
Copy link
Copy Markdown
Member

Follow up to #34662 that moves almost all logic to progress-bar.js.

image

Auto progresses ad and bar shows 8 second animation. Will be ran as an experiment to determine UX improvements.

Partial #33969

@amp-owners-bot
Copy link
Copy Markdown

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

extensions/amp-story/1.0/amp-story-system-layer.css
extensions/amp-story/1.0/progress-bar.js
extensions/amp-story/1.0/test/test-amp-story-progress-bar.js

@calebcordry calebcordry enabled auto-merge (squash) June 10, 2021 22:41
@calebcordry calebcordry merged commit 51f5ac1 into ampproject:main Jun 10, 2021
@calebcordry calebcordry deleted the yellow-v2 branch June 10, 2021 23:16
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants