#54284 closed defect (bug) (fixed)
CSS coding standards issue in the `print_emoji_styles()` function
| Reported by: | audrasjb | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.9 |
| Component: | Formatting | Version: | |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: | css |
Description
There is an unwanted space in the margin declaration of the styles printed by the print_emoji_styles() function.
Attachments (2)
Change History (11)
#1
@
5 years ago
- Keywords has-patch added; needs-patch removed
In 54284.diff:
Coding Standards: Formatting: Remove an unwanted space in the CSS printed by the print_emoji_styles() function.
#2
@
5 years ago
I think the margin is only intended for the sides (it had been 0 .05em 0 .1em before r31786).
However, we still could add the leading zero before the decimal point:
margin: 0 0.07em !important;
#3
@
5 years ago
- Keywords needs-refresh added
Hooooo good point, I think I was confused because the leading zero was missing :D
Editing the patch accordingly
#4
@
5 years ago
Refreshed in 54284.1.diff
Indeed, the leading zero suggested by WPCS is less confusing :)
#6
@
5 years ago
New explanation suggestion for the changeset:
Formatting: Coding Standards: Add a leading zero in the CSS declarations printed by the print_emoji_styles() function.
#7
@
5 years ago
- Keywords commit added
54284.1.diff patch looks good to me.
Ready to marge, mark as commit.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Coding Standards: Formatting: Remove an unwanted space in the
print_emoji_styles()function.