Skip to content
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

Consider making twentyseventeen_is_static_front_page() canonical #23

Closed
wants to merge 2 commits into from

Conversation

sabernhardt
Copy link
Owner

  • Updates the twentyseventeen_is_frontpage() to use twentyseventeen_is_static_front_page()
  • Uses twentyseventeen_is_static_front_page() for all instances in the theme
  • Adds a few extra documentation changes, which may fit better in the general Docblock updates ticket for 6.3

Trac ticket: https://core.trac.wordpress.org/ticket/43515


This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

@@ -69,7 +69,7 @@ function twentyseventeen_body_classes( $classes ) {
/**
* Count our number of active panels.
*
* Primarily used to see if we have any panels active, duh.
* Primarily used to see if we have any panels active.

Choose a reason for hiding this comment

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

Nice catch!

@sabernhardt sabernhardt deleted the 2017-frontpage-functions branch August 17, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants