Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

use select field for ratings if IE11 #81

Merged
merged 3 commits into from
May 8, 2018
Merged

Conversation

DavidCramer
Copy link
Contributor

Corrects the star rating on IE11 as mentioned in #8

@DavidCramer DavidCramer requested a review from kienstra May 7, 2018 07:37
@DavidCramer
Copy link
Contributor Author

@kienstra You can give this a test. Thanks.

Copy link
Contributor

@kienstra kienstra left a comment

Choose a reason for hiding this comment

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

Works Well, Very Minor Point

Hi David,
Thanks for this good solution to IE 11 support. And thanks for reviewing #80 right away, after your workday.

There's a very minor point here.

Ratings on the single 'adventure' page now work on IE 11:
ie-11-rating

* Add select rating field to comments.
*/
function amp_travel_comment_select_rating_field() {
echo '<p class="comment-form-rating-select"><label for="rating">' . esc_html( 'Your rating' ) . '</label>
Copy link
Contributor

Choose a reason for hiding this comment

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

Could this be:

esc_html__( 'Your rating', 'travel' )

@DavidCramer
Copy link
Contributor Author

@kienstra Thanks, changes applied and conflicts resolved.

@kienstra
Copy link
Contributor

kienstra commented May 8, 2018

Thanks, @DavidCramer! This looks good.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants