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
Implement amp-story tooltip UI #16516
Comments
This is a high priority issue but it hasn't been updated in awhile. @Enriqe Do you have any updates? |
We are in design phase for this, implementation should start right after. |
This is a high priority issue but it hasn't been updated in awhile. @Enriqe Do you have any updates? |
1 similar comment
This is a high priority issue but it hasn't been updated in awhile. @Enriqe Do you have any updates? |
@Enriqe do we have a separate issue to track unit and visual diff tests? |
@newmuis Sure, I'll create one for the tests and validation. |
Given an element as its anchor, we should have a tooltip UI that looks like:
The icon and string should both be specified as attributes from the anchor element. The string can have a runtime-specified default (e.g. "Tap to view"). The action icon should be specified by the runtime, depending on the action (e.g. open in lightbox, navigate to URL). For example:
The chevron should always be centered on the anchor element. The bubble should also be centered on the anchor element, if possible, otherwise it should be as close to centered as possible within the bounds of the viewport.
The text was updated successfully, but these errors were encountered: