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-cta-layer I2D #34450

Open
processprocess opened this issue May 19, 2021 · 1 comment
Open

amp-story-cta-layer I2D #34450

processprocess opened this issue May 19, 2021 · 1 comment
Assignees
Labels
INTENT TO DEPRECATE Proposes deprecating an existing AMP feature. Stale Inactive for one year or more WG: stories

Comments

@processprocess
Copy link
Contributor

processprocess commented May 19, 2021

Summary

The <amp-story-cta-layer> allows the placement of multiple CTAs on the bottom of the page but in practice very few Stories make use of this freedom. In almost all cases, Stories use a single CTA at the bottom of the page. Further, it is easy to create poor user experience situations using the cta-layer (e.g. a wide invisible tap target at the bottom of the page).
In order to make outlinking more consistent and predictable for users we intend to depreciate and remove <amp-story-cta-layer> in favor of <amp-story-page-outlink>.

amp-story-page-outlink:
Screen Shot 2021-05-19 at 3 56 24 PM

Motivation

Depreciating and removing the cta-layer in favor of amp-story-page-outlink will ensure a more consistent and expected UX for users.

Impact on Existing Users

Any existing story that uses the cta-layer will visually look different since the custom styled element will be removed and replaced with an amp-story-page-outlink element.

A benefit of this is that it requires no action on the publisher and the outlinking UX will still be the one-tap UX.

Depreciation process

After a 3 month depreciation time behind an experiment we will convert all cta-layers to amp-story-page-outlink elements.
The experiment can be manually turned on to test existing stories.

This change will be automatic and will not require any action from publishers. As a result any existing stories using the cta-layer will have a benefit of consistent UX with amp-story-page-outlink.
To create the amp-story-page-outlink the cta-layer will be parsed to find the link and its href/anchor.

If cta-layer contains one anchor element:
The href will be used to build the amp-story-page-outlink that replaces the cta-layer. If the anchor element contains text, the text will be applied to the amp-story-page-outlink button.
amp-story-page-outlink wraps an a tag element.

If cta-layer contains two anchor elements:
The elements will trigger a tooltip like other page links.

Notifications

/cc @ampproject/wg-approvers

@stale
Copy link

stale bot commented Jan 21, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the Stale Inactive for one year or more label Jan 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INTENT TO DEPRECATE Proposes deprecating an existing AMP feature. Stale Inactive for one year or more WG: stories
Projects
None yet
Development

No branches or pull requests

1 participant