-
Notifications
You must be signed in to change notification settings - Fork 13
Issues #75, #76, #77: fix various front-end issues. #78
Conversation
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.
@delawski this looks good, but please update the on
action to be on the button
, not the parent div
.
comments.php
Outdated
<button on="tap:amp-live-comments-list-<?php the_ID(); ?>.update"> | ||
<?php esc_html_e( 'New comment(s)', 'ampconf' ); ?> | ||
</button> | ||
<div update on="tap:amp-live-comments-list-<?php the_ID(); ?>.update" class="live-list__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.
The on
action here should be on the button
element, not the parent div
.
home.php
Outdated
<amp-live-list id="ampconf-featured-articles-list" data-poll-interval="<?php echo esc_attr( AMPCONF_LIVE_LIST_POLL_INTERVAL ); ?>" data-max-items-per-page="3"> | ||
<button class="button" update on="tap:ampconf-articles-list.update,ampconf-featured-articles-list.update"><?php esc_html_e( 'Load Newer Articles', 'ampconf' ); ?></button> | ||
<amp-live-list id="ampconf-featured-articles-list" class="live-list" data-poll-interval="<?php echo esc_attr( AMPCONF_LIVE_LIST_POLL_INTERVAL ); ?>" data-max-items-per-page="3"> | ||
<div class="live-list__button" update on="tap:ampconf-articles-list.update,ampconf-featured-articles-list.update"> |
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.
The on
action here should be on the button
element, not the parent div
.
home.php
Outdated
@@ -70,7 +72,9 @@ | |||
?> | |||
</div> | |||
|
|||
<button class="button" update on="tap:ampconf-articles-list.update<?php echo esc_attr( ( ! is_paged() ) ? ',ampconf-featured-articles-list.update' : '' ); ?>"><?php esc_html_e( 'Load Newer Articles', 'ampconf' ); ?></button> | |||
<div class="live-list__button" update on="tap:ampconf-articles-list.update<?php echo esc_attr( ( ! is_paged() ) ? ',ampconf-featured-articles-list.update' : '' ); ?>"> |
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.
The on
action here should be on the button
element, not the parent div
.
page.php
Outdated
<amp-live-list id="ampconf-live-page-<?php the_ID(); ?>" data-poll-interval="<?php echo esc_attr( AMPCONF_LIVE_LIST_POLL_INTERVAL ); ?>" data-max-items-per-page="1"> | ||
<button update on="tap:ampconf-live-page-<?php the_ID(); ?>.update" class="button"><?php esc_html_e( 'Load updates!', 'ampconf' ); ?></button> | ||
<amp-live-list id="ampconf-live-page-<?php the_ID(); ?>" class="live-list" data-poll-interval="<?php echo esc_attr( AMPCONF_LIVE_LIST_POLL_INTERVAL ); ?>" data-max-items-per-page="1"> | ||
<div update on="tap:ampconf-live-page-<?php the_ID(); ?>.update" class="live-list__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.
The on
action here should be on the button
element, not the parent div
.
single.php
Outdated
<amp-live-list id="ampconf-live-post-<?php the_ID(); ?>" data-poll-interval="<?php echo esc_attr( AMPCONF_LIVE_LIST_POLL_INTERVAL ); ?>" data-max-items-per-page="1"> | ||
<button update on="tap:ampconf-live-post-<?php the_ID(); ?>.update" class="button"><?php esc_html_e( 'Load updates!', 'ampconf' ); ?></button> | ||
<amp-live-list id="ampconf-live-post-<?php the_ID(); ?>" class="live-list" data-poll-interval="<?php echo esc_attr( AMPCONF_LIVE_LIST_POLL_INTERVAL ); ?>" data-max-items-per-page="1"> | ||
<div update on="tap:ampconf-live-post-<?php the_ID(); ?>.update" class="live-list__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.
The on
action here should be on the button
element, not the parent div
.
templates/live-lists/posts.php
Outdated
<amp-live-list id="ampconf-posts-list" data-poll-interval="<?php echo esc_attr( AMPCONF_LIVE_LIST_POLL_INTERVAL ); ?>" data-max-items-per-page="<?php echo esc_attr( get_option( 'posts_per_page' ) ); ?>"> | ||
<button update on="tap:ampconf-posts-list.update" class="button"><?php esc_html_e( 'Load Newer Articles', 'ampconf' ); ?></button> | ||
<amp-live-list id="ampconf-posts-list" class="live-list" data-poll-interval="<?php echo esc_attr( AMPCONF_LIVE_LIST_POLL_INTERVAL ); ?>" data-max-items-per-page="<?php echo esc_attr( get_option( 'posts_per_page' ) ); ?>"> | ||
<div update on="tap:ampconf-posts-list.update" class="live-list__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.
The on
action here should be on the button
element, not the parent div
.
templates/more-stories.php
Outdated
<?php esc_html_e( 'Load Newer Stories', 'ampconf' ); ?> | ||
</button> | ||
<amp-live-list id="more-stories" class="live-list" data-poll-interval="<?php echo esc_attr( AMPCONF_LIVE_LIST_POLL_INTERVAL ); ?>" data-max-items-per-page="<?php echo esc_attr( $max_shown ); ?>"> | ||
<div update on="tap:more-stories.update" class="live-list__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.
The on
action here should be on the button
element, not the parent div
.
@westonruter |
Changes have been deployed. |
Fixes #78.
Fixes #76.
Fixes #77.