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

Genesis Sample 3.0.0 release #268

Merged
merged 47 commits into from
Jun 19, 2019
Merged

Genesis Sample 3.0.0 release #268

merged 47 commits into from
Jun 19, 2019

Conversation

nickcernis
Copy link
Collaborator

3.0.0 scheduled for 2019-06-19

Requires Genesis 3.0.0+.

Added

  • AMP support. (Optional, off by default, requires Genesis 3.0.0 and the AMP plugin.)
  • New composer run phpcbf command to fix simple PHP linting violations. (GitHub version only.)

Changed

  • Use position: sticky for the header and remove JavaScript that adjusted header height and content position. This prevents content reflow after page load.
  • Reorganized config folder for easier reading and maintenance.
  • Use new genesis_get_theme_handle() and genesis_get_theme_version() functions from Genesis 3.0.0 in place of declaring CHILD_THEME_HANDLE and CHILD_THEME_VERSION constants.
  • Update developer dependencies.

Fixed

  • Improved block column CSS for WordPress 5.2+.

Removed

  • Responsive menu scripts have been replaced with genesis_register_responsive_menus(), which loads required JavaScript from Genesis 3.0.0.
  • genesis-responsive-viewport theme support declaration. This is now added by default in Genesis 3.0.0.
  • Deleted Genesis nav extras filter and metabox removal. These are no longer present in Genesis 3.0.0.
  • child-theme-settings-genesis.php file and filter. Genesis 3.0.0 no longer includes a settings admin page or genesis_theme_settings_defaults filter.
  • 404-page accessibility support, as this no longer has an effect.

nickcernis and others added 30 commits May 1, 2019 08:27
Creating a working form via the WPForms API requires 1.5.2 or higher.

If the site already has WPForms Lite installed but it's an older
version, form creation will fail, breaking the onboarding process.
This is no longer used in Genesis, so has no effect.
Loads these files from the Genesis AMP plugin (relocated to the theme):

- theme-fixer/genesis-amp-animated.css
- theme-fixer/themes/genesis-sample/stylesheet.css (omits .woocommerce)

This allows the menu to function on AMP and regular versions when
used with this version of the Genesis AMP plugin that omits all
'theme fixer' functionality:

https://github.com/nickcernis/genesis-amp/tree/suppress-theme-fixer

Still need to:

- Discuss if genesis-amp-animated.css belongs in Genesis or the theme.
- Look into not loading a separate stylesheet conditionally for AMP. It
would be easier to maintain if the AMP menu CSS was part of the theme's
main style.css.
These are not needed for modern AMP-ready browsers.
Ensures submenus appear on hover instead of only on focus.

Props @dreamwhisper.
This will be served from Genesis itself from
Genesis 3.0.0+ and enqueued automatically when using
the `genesis_register_responsive_menus` function.
Support for the viewport meta tag is added by default in Genesis 3.0
Since 3.0 removes settings pages and Sample 3.0 requires Genesis 3.0,
remove the edits we had in place to remove specific metaboxes.
Genesis 3.0+ does not have a theme settings admin page or
`genesis_theme_settings_defaults` filter, so I've removed that filter
from theme-defaults.php.

As the Genesis settings are then no longer needed in two places,
I moved settings from `child-theme-settings-genesis.php` to the main
`child-theme-settings.php` file.
…reset

Remove Genesis-specific theme settings files
We can retrieve the default color from the appearance config instead.

We don't have getter functions to retrieve other preferences, so it
seems unnecessary to have one for the colors alone.
@dreamwhisper dreamwhisper merged commit 4a1a40a into master Jun 19, 2019
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.

3 participants