-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Add Bento documentation for amp-date-countdown #32256
Conversation
Bento docs for amp-date-countdown
@CrystalOnScript or @caroqliu for review. Thank you! |
</template> | ||
</amp-date-countdown> | ||
<div class="buttons" style="margin-top: 8px;"> | ||
<button id="ar-button">Change locale to Arabic</button> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like amp-date-countdown doesn't support Arabic: https://amp.dev/documentation/components/amp-date-countdown/?format=websites#locale-(optional) please replace here and in the click event with a new value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed!
layout="fixed-height" | ||
height="50"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dvoytenko This sample complains when you remove layout
and height
FYI
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(We should remove them if we can btw)
Co-authored-by: Caroline Liu <10456171+caroqliu@users.noreply.github.com>
layout="fixed-height" | ||
height="50"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(We should remove them if we can btw)
Co-authored-by: Caroline Liu <10456171+caroqliu@users.noreply.github.com>
* Bento docs for amp-date-countdown Bento docs for amp-date-countdown * Apply suggestions from code review Co-authored-by: Caroline Liu <10456171+caroqliu@users.noreply.github.com> * Fix copy pasta errors * Update extensions/amp-date-countdown/amp-date-countdown.md Co-authored-by: Caroline Liu <10456171+caroqliu@users.noreply.github.com> * Travis pliss pass * prettify Co-authored-by: Caroline Liu <10456171+caroqliu@users.noreply.github.com>
Add Bento documentation for amp-date-countdown