Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#25195 closed defect (bug) (fixed)

Twenty Thirteen: mobile comment layout can break if a really, really long word is in the comment content

Reported by: lancewillett's profile lancewillett Owned by: lancewillett's profile lancewillett
Milestone: 3.7 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

Reported by Mark Jaquith via Twitter: https://twitter.com/markjaquith/status/373426577151102976

Example URL: http://www.viper007bond.com/2013/08/27/date-queries-in-wordpress-3-point-7/comment-page-1/#comment-237786

Probably a word-wrap: break-word; for comment-content element (and maybe just in small screens).

Attachments (4)

IMG_2712.jpg (254.4 KB) - added by lancewillett 11 years ago.
IMG_2713.jpg (324.2 KB) - added by lancewillett 11 years ago.
25195.diff (422 bytes) - added by RDall 11 years ago.
Adds word-wrap break-word to the comment-content in the media query for tablets
25195.1.diff (400 bytes) - added by obenland 11 years ago.

Download all attachments as: .zip

Change History (9)

#1 @helen
11 years ago

#25199 was marked as a duplicate.

@RDall
11 years ago

Adds word-wrap break-word to the comment-content in the media query for tablets

#2 @RDall
11 years ago

  • Keywords has-patch added; needs-patch removed

@obenland
11 years ago

#3 @lancewillett
11 years ago

Rule needs to be outside of media query so it applies in all layouts.

#4 @lancewillett
11 years ago

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

In 25516:

Twenty Thirteen: fix case where comment layout can break if a really, really long word is in the comment content. Closes #25195 props RDall and obenland.

#5 @rdall
11 years ago

Shoot guys if I had known we wanted to cover all media queries I would have placed there. :-)

Note: See TracTickets for help on using tickets.