Opened 5 years ago
Closed 5 years ago
#48592 closed defect (bug) (fixed)
About display of tag cloud widget
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.3.1 | Priority: | normal |
Severity: | normal | Version: | 5.3 |
Component: | Editor | Keywords: | has-screenshots has-patch commit fixed-major |
Focuses: | ui | Cc: |
Description
The classic editor tag cloud is not displayed correctly.
Specifically, the line spacing between tags has increased, making usability worse.
In version 5.3, “#poststuff .inside .the-tagcloud” has been changed from media.css to edit.css.
At that time, the value of line-height has been changed from 18em to 18.
/wp-admin/css/media.css(5.2.4)
#poststuff .inside .the-tagcloud { margin: 5px 0 10px; padding: 8px; border: 1px solid #ddd; line-height: 1.8em; word-spacing: 3px; }
/wp-admin/css/edit.css(5.3.0)
#poststuff .inside .the-tagcloud { margin: 5px 0 10px; padding: 8px; border: 1px solid #ddd; line-height: 1.8; word-spacing: 3px; }
Setting the line-height value between 1.1 and 1.2 will resemble the traditional display.
Please adjust this value.
Attachments (7)
Change History (16)
#3
@
5 years ago
- Focuses ui added
- Owner set to audrasjb
- Status changed from new to accepted
Hi,
I tested both 1.2
and 1.1
values and I'd recommend 1.2 which is closer to the previous styles.
See screenshots above.
Here is the suggested patch on edit.css:
#poststuff .inside .the-tagcloud { margin: 5px 0 10px; padding: 8px; border: 1px solid #ddd; line-height: 1.2; word-spacing: 3px; }
#4
@
5 years ago
- Keywords needs-refresh added
Thanks for the patch @sathyapulse
However, the patch doesn't apply cleanly.
You have to generate it from wordpress-develop.
Adding needs-refresh
keyword.
#5
@
5 years ago
- Keywords has-patch commit added; needs-refresh removed
Thanks @Spaceshipone @sathyapulse both of the last 48592.1
patch work on my side!
Thank you very much!
Adding commit
keyword :-)
#6
@
5 years ago
For reference, on 48592.png: the tag cloud in 5.2 (left) and with the proposed patch (right).
The one from 5.2 with fixed height (1.8em) looks a bit more balanced to me on lines with smaller font size, however the proposed one seems close enough, and is an improvement over the current styles. Should be good to go.
Introduced in [45483].