-
Notifications
You must be signed in to change notification settings - Fork 36
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
Promotional updates #27
Conversation
zbtirrell
commented
Mar 17, 2017
- update readme
- update admin notices
- update description on plugins page
image-widget.php
Outdated
@@ -438,7 +447,7 @@ public function post_upgrade_nag() { | |||
*/ | |||
public function plugin_row_meta( $meta, $file ) { | |||
if ( $file == plugin_basename( dirname( __FILE__ ) . '/image-widget.php' ) ) { | |||
$meta[] = '<span class="tribe-test">' . sprintf( __( 'Check out our other <a href="%s" target="_blank">plugins</a> including our <a href="%s" target="_blank">Events Calendar Pro</a>!', 'image-widget' ), 'https://theeventscalendar.com/products/?source=image-widget&pos=pluginlist', 'https://theeventscalendar.com/product/wordpress-events-calendar-pro/?source=image-widget&pos=pluginlist' ) . '</span>'; | |||
$meta[] = '<strong>Coming Soon:</strong> <a href="http://m.tri.be/19ma" target="_blank">' . __( 'Image Widget Plus', 'image-widget' ) . '</a>'; |
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.
esc_html__()
for Image Widget Plus?- Add
esc_html__()
for "Coming Soon"
image-widget.php
Outdated
<div class="update-nag"> | ||
<?php esc_html_e( 'Thanks for using the Image Widget by Modern Tribe! If you like these features, you\'ll love what we\'re working on next.', 'image-widget' );?> | ||
<br> | ||
<?php echo sprintf( |
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.
Not essential, but printf()
is a little more concise then echo sprintf()
image-widget.php
Outdated
esc_html__( 'Check out the new %1$sImage Widget Plus%2$s!', 'image-widget' ), | ||
'<a href="http://m.tri.be/19mb" target="_blank">', | ||
'</a>' | ||
);?> |
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.
Space after ;
readme.txt
Outdated
[Check us out on GitHub](https://github.com/moderntribe/image-widget) in case you want to add some pull requests or a fork! | ||
= Pull Requests & Translations = | ||
|
||
to add your pull requests, translations, or a fork. Translations can also be submitted in [our forum here on WordPress.org](https://wordpress.org/support/plugin/image-widget/). |
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.
Start this sentence with a capital letter?
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.
Also, seems a little weird (to me) to have "Translations can also be submitted..." here and below the "Check us out on GitHub" link -- the second link seems preferable to me as it goes straight to the translations project.
this is ready for review again |
👍 |