#56675 closed defect (bug) (fixed)
Extra spaces appear in css queue
Reported by: | elten | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.1 | Priority: | normal |
Severity: | minor | Version: | |
Component: | Script Loader | Keywords: | has-patch has-unit-tests commit |
Focuses: | coding-standards | Cc: |
Description
In the css queue, there are two spaces between “id” and “href”, also have this problem in wordpress.org.
<link rel='stylesheet' id='wporg-global-fonts-css' href='https://wordpress.org/wp-content/mu-plugins/pub-sync/global-fonts/style.css?ver=1663691227' media='all' /> <link rel='stylesheet' id='wporg-parent-2021-style-css' href='https://wordpress.org/wp-content/themes/wporg-parent-2021/build/style.css?ver=1660938306' media='all' /> <link rel='stylesheet' id='wporg-main-2022-style-css' href='https://wordpress.org/wp-content/themes/wporg-main-2022/build/style/style-index.css?ver=1663690308' media='all' /> <link rel='stylesheet' id='wporg-parent-block-styles-css' href='https://wordpress.org/wp-content/themes/wporg-parent-2021/build/block-styles.css?ver=1661271656' media='all' /> <link rel='stylesheet' id='wporg-global-header-footer-css' href='https://wordpress.org/wp-content/mu-plugins/pub-sync/blocks/global-header-footer/build/style.css?ver=1664222912' media='all' />
==========
WordPress version : 6.0.2
Appears after loading the css queue ( use wp_enqueue_style() ) .
The problem has nothing to do with templates.
Attachments (1)
Change History (10)
This ticket was mentioned in PR #3354 on WordPress/wordpress-develop by whaze.
2 years ago
#2
- Keywords has-patch added; needs-patch removed
Trac ticket: https://core.trac.wordpress.org/ticket/56675
#4
@
2 years ago
- Keywords changes-requested added
Looks great to me.
By the way, we should also update the related unit tests located in tests/phpunit/tests/dependencies/styles.php
@whaze
#6
@
2 years ago
- Keywords has-unit-tests commit added; changes-requested removed
- Milestone changed from Awaiting Review to 6.1
- Version 6.0.2 deleted
Thanks @whaze, looks great.
Also, thanks @SergeyBiryukov for adding RTL support and for the few small enhancements ✌️
Let's ship this with 6.1.
#8
@
2 years ago
- Owner set to audrasjb
- Resolution set to fixed
- Status changed from new to closed
In 54348:
2 years ago
#9
committed in https://core.trac.wordpress.org/changeset/54348
Hello, welcome to WordPress Core Trac and thank you for opening this ticket!
The issue comes from this line. When the third
%s
placeholder is empty, two spaces are printed.