We want to highlight the latest Safari Technology Preview release notes when they are the current post, but otherwise, show more of our other blog content.
Created attachment 310667 [details] Patch
Comment on attachment 310667 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=310667&action=review Looks sane to me > Websites/webkit.org/ChangeLog:8 > + * wp-content/themes/webkit/functions.php: Load some extra posts Nit: Missing period at the end of this sentence. > Websites/webkit.org/ChangeLog:9 > + * wp-content/themes/webkit/widgets/post.php: Show STP blog posts when it is the current featured post Ditto. > Websites/webkit.org/wp-content/themes/webkit/functions.php:420 > + Please remove this whitespace. > Websites/webkit.org/wp-content/themes/webkit/widgets/post.php:35 > + if ( ! $featured) { Nit: Missing space before the ')'. > Websites/webkit.org/wp-content/themes/webkit/widgets/post.php:36 > + // Prevent Safari Technology Preview release note posts from showing up when not a featured post Nit: Missing a period at the end of this sentence.
Created attachment 311612 [details] Patch
Comment on attachment 311612 [details] Patch Clearing flags on attachment: 311612 Committed r217614: <http://trac.webkit.org/changeset/217614>
All reviewed patches have been landed. Closing bug.