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

📖 [Amp story] [Page attachments] Amp.dev Docs for New Page Attachment Features #34883 #35338

Merged
merged 31 commits into from Aug 2, 2021

Conversation

processprocess
Copy link
Contributor

Updates amp-story-page-attachment docs
Adds amp-story-page-outlink docs

Fixes #34790

@processprocess processprocess marked this pull request as ready for review July 21, 2021 19:39
@amp-owners-bot
Copy link

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

extensions/amp-story/amp-story-page-attachment.md
extensions/amp-story/amp-story-page-outlink.md
extensions/amp-story/img/amp-story-page-attachment-1-image.jpg
extensions/amp-story/img/amp-story-page-attachment-2-images.jpg
extensions/amp-story/img/amp-story-page-attachment-cta-text.jpg
extensions/amp-story/img/amp-story-page-attachment-dark-theme.jpg
extensions/amp-story/img/amp-story-page-attachment-default.jpg
extensions/amp-story/img/amp-story-page-attachment.gif
extensions/amp-story/img/amp-story-page-outlink-cta-image-none.jpg
extensions/amp-story/img/amp-story-page-outlink-cta-image.jpg
extensions/amp-story/img/amp-story-page-outlink-cta-text.jpg
extensions/amp-story/img/amp-story-page-outlink-custom-theme-background.jpg
+4 more

@processprocess processprocess added this to In progress in wg-stories Sprint via automation Aug 2, 2021
@processprocess processprocess added this to In progress in wg-stories Page attachments via automation Aug 2, 2021
@processprocess processprocess removed this from In progress in wg-stories Sprint Aug 2, 2021
@processprocess processprocess removed this from In progress in wg-stories Page attachments Aug 2, 2021
Copy link
Contributor

@CrystalOnScript CrystalOnScript left a comment

Choose a reason for hiding this comment

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

Approved - with one nit to change the use of the word "suggested". Thanks!

extensions/amp-story/amp-story-page-attachment.md Outdated Show resolved Hide resolved
extensions/amp-story/amp-story-page-attachment.md Outdated Show resolved Hide resolved
extensions/amp-story/amp-story-page-outlink.md Outdated Show resolved Hide resolved
@processprocess processprocess merged commit c243a8b into ampproject:main Aug 2, 2021
@processprocess processprocess deleted the docs branch August 2, 2021 17:49
westonruter added a commit to westonruter/amphtml that referenced this pull request Aug 3, 2021
…tok-validation

* 'main' of github.com:ampproject/amphtml: (72 commits)
  build: run amp lint --fix to address import order of jixie (ampproject#35513)
  ✨ [amp-analytics] Add Custom Browser Event Tracker (ampproject#35193)
  babel: teach amp mode transformer about #core/mode (ampproject#35477)
  🚮 Remove experiment `amp-consent-granular-consent` (ampproject#35508)
  ♻️ Enable auto-sorting+grouping within src/ and 3p/ (ampproject#35454)
  🐛  [amp-render] fix root-element stripping from amp-render with amp-bind (ampproject#35449)
  ✅ [Story interactive] Add Example Story for Detailed Results Component (ampproject#35450)
  🐛 Fix error on bento example (ampproject#35490)
  🐛 amp-story-grid-layer: Fix AMP invalidation error in documentation (ampproject#35503)
  🐛 Fix code formatting (ampproject#35499)
  ✅ buildDom: add tests for amp-fit-text and amp-layout (ampproject#35494)
  ♻️ Remove unused imports (ampproject#35435)
  ✨ amp-connatix-player: iframe domain based on a property (ampproject#35179)
  Updated document with use cases of remote config (ampproject#35496)
  AMP.goBack: update documentation (ampproject#29290)
  🏗 Allow the bundle-size job to run even if the builds were skipped (ampproject#35492)
  build-system: improve terser/esbuild integration (ampproject#35466)
  🧪 [Story performance] Disable animations on first page to 50% (ampproject#35476)
  📖 [Amp story] [Page attachments] Amp.dev Docs for New Page Attachment Features ampproject#34883 (ampproject#35338)
  🚀 [Story interactive] Rewrite Image URL to Cached URL for Image Quizzes and Polls (ampproject#35375)
  ...
@MCFreddie777
Copy link

@processprocess I think you removed ## Linked content section by an accident.

@processprocess
Copy link
Contributor Author

@processprocess I think you removed ## Linked content section by an accident.

Hi @MCFreddie777 :)
We created the amp-story-page-outlink component for linking behavior. We did this so that amp-story-page-attachment can handle all inline attachment logic and amp-story-page-outlink can handle all outlink logic.
We will be posting a blogpost soon that explains all of this in more detail.
Sorry for any confusion!

@processprocess
Copy link
Contributor Author

@MCFreddie777 The blogpost with details on the CTA updates is available here :)

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.

[Page Attachments] Write amp.dev docs
5 participants