Bug 172364 – Reduce Safari Technology Preview Release Notes posts shown on homepage

• Component: WebKit Website
• Status: RESOLVED FIXED

[Jon Davis]

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.

[Jon Davis]

Created attachment 310667 [details]
Patch

[Daniel Bates]

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.

[Jon Davis]

Created attachment 311612 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 311612 [details]
Patch

Clearing flags on attachment: 311612

Committed r217614: <http://trac.webkit.org/changeset/217614>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.