Opened 6 months ago
Closed 6 months ago
#60779 closed enhancement (fixed)
Twenty Twenty: use $theme_version variable for font stylesheet
Reported by: | sabernhardt | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 6.6 | Priority: | normal |
Severity: | minor | Version: | |
Component: | Bundled Theme | Keywords: | has-patch |
Focuses: | Cc: |
Description
When [57311] added a font stylesheet,
wp_get_theme()->get( 'Version' )
identified the theme version within wp_enqueue_style()
.
twentytwenty_register_styles()
already has a$theme_version
variable for use with other stylesheets, and it could be reused.twentytwenty_block_editor_styles()
is the only function that includeswp_get_theme()->get( 'Version' )
without declaring the variable, and now the function uses it twice.twentytwenty_register_scripts()
andtwentytwenty_customize_preview_init()
have the variable but only use it once each.
Attachments (1)
Change History (4)
#1
@
6 months ago
- Keywords has-patch added
For 60779.diff, I chose consistency. With that, all five of the enqueuing functions declare and use the $theme_version
variable.
Note: See
TracTickets for help on using
tickets.
uses $theme_version variable in twentytwenty_register_styles() and adds the variable in twentytwenty_block_editor_styles()