WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 2 years ago

#17424 closed defect (bug) (fixed)

Long text in Recent Drafts widget

Reported by: Lardjo Owned by: azaozz
Milestone: 3.4 Priority: normal
Severity: trivial Version: 3.2
Component: General Keywords: has-patch commit
Focuses: Cc:

Description (last modified by SergeyBiryukov)

Recent Drafts widget has a small bug. If a post title is too long (you never can tell), then it goes out of the widget box:
17424-before.png

This problem exists in all recent versions. We can fix it with some changes to wp-admin/css/dashboard.css:

#dashboard_recent_drafts h4 {
	line-height: 1.7em;
	word-wrap: break-word;
}

#dashboard_recent_drafts p {
	margin: 0;
	padding: 0;
	word-wrap: break-word;
}

The result would be much better:
17424-after.png

Why not fix it? ))

Attachments (5)

dashboard.css (6.2 KB) - added by Lardjo 3 years ago.
dashboard.dev.patch (351 bytes) - added by Lardjo 3 years ago.
17424.patch (475 bytes) - added by SergeyBiryukov 3 years ago.
17424-before.png (5.8 KB) - added by SergeyBiryukov 3 years ago.
17424-after.png (6.3 KB) - added by SergeyBiryukov 3 years ago.

Download all attachments as: .zip

Change History (16)

Lardjo3 years ago

comment:1 follow-up: Denis-de-Bernardy3 years ago

Well... at least the screenshots make the bug understandable. :-D

comment:2 in reply to: ↑ 1 ; follow-up: Lardjo3 years ago

Replying to Denis-de-Bernardy:

Well... at least the screenshots make the bug understandable. :-D

It was hard to write all in English)) Pictures were made ​​to understand.

comment:3 in reply to: ↑ 2 ; follow-up: Denis-de-Bernardy3 years ago

Replying to Lardjo:

Replying to Denis-de-Bernardy:

Well... at least the screenshots make the bug understandable. :-D

It was hard to write all in English)) Pictures were made ​​to understand.

Ideally create a diff/patch file instead of uploading the file, though. It'll make the changes more clear.

comment:4 in reply to: ↑ 3 Lardjo3 years ago

Replying to Denis-de-Bernardy:

Replying to Lardjo:

Replying to Denis-de-Bernardy:

Well... at least the screenshots make the bug understandable. :-D

It was hard to write all in English)) Pictures were made ​​to understand.

Ideally create a diff/patch file instead of uploading the file, though. It'll make the changes more clear.

Yes. I know. But CSS is made in a single line, so the patch is bad. Ideally, immediately replace all the CSS, which I attached. But if you want, I can make a .diff file.

comment:5 xknown3 years ago

Don't use the minified version, use dashboard.dev.css to make the patch.

Lardjo3 years ago

SergeyBiryukov3 years ago

SergeyBiryukov3 years ago

SergeyBiryukov3 years ago

comment:6 SergeyBiryukov3 years ago

  • Description modified (diff)
  • Summary changed from Length text in widget to Long text in Recent Drafts widget

The original screenshots are now gone, so I've uploaded the new ones to make the bug understandable again and translated the description into English. Also refreshed the patch to include proper path.

We already have word-wrap: break-word; for .dashboard-comment-wrap, introduced in #9283.

comment:7 Lardjo3 years ago

Finally. My ticket took up:) Just been a long time - screenshots deleted. :(
I tied the patch and the CSS file. I hope they will help.

comment:8 SergeyBiryukov3 years ago

  • Keywords has-patch added

comment:9 nacin2 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to Future Release

Looks good.

comment:10 azaozz2 years ago

  • Component changed from Widgets to General
  • Milestone changed from Future Release to 3.4

comment:11 azaozz2 years ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In [19694]:

Wrap long titles in dashboard recent drafts, props Lardjo SergeyBiryukov, fixes #17424

Note: See TracTickets for help on using tickets.