WordPress.org

Make WordPress Core

Opened 10 years ago

Closed 9 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 10 years ago.
dashboard.dev.patch (351 bytes) - added by Lardjo 10 years ago.
17424.patch (475 bytes) - added by SergeyBiryukov 9 years ago.
17424-before.png (5.8 KB) - added by SergeyBiryukov 9 years ago.
17424-after.png (6.3 KB) - added by SergeyBiryukov 9 years ago.

Download all attachments as: .zip

Change History (16)

@Lardjo
10 years ago

#1 follow-up: @Denis-de-Bernardy
10 years ago

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

#2 in reply to: ↑ 1 ; follow-up: @Lardjo
10 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.

#3 in reply to: ↑ 2 ; follow-up: @Denis-de-Bernardy
10 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.

#4 in reply to: ↑ 3 @Lardjo
10 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.

#5 @xknown
10 years ago

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

#6 @SergeyBiryukov
9 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.

#7 @Lardjo
9 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.

#8 @SergeyBiryukov
9 years ago

  • Keywords has-patch added

#9 @nacin
9 years ago

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

Looks good.

#10 @azaozz
9 years ago

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

#11 @azaozz
9 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.