WordPress.org

Make WordPress Core

Opened 10 months ago

Closed 10 months ago

Last modified 10 months 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 Owned by: 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 10 months ago.
IMG_2713.jpg (324.2 KB) - added by lancewillett 10 months ago.
25195.diff (422 bytes) - added by RDall 10 months ago.
Adds word-wrap break-word to the comment-content in the media query for tablets
25195.1.diff (400 bytes) - added by obenland 10 months ago.

Download all attachments as: .zip

Change History (9)

lancewillett10 months ago

lancewillett10 months ago

comment:1 helen10 months ago

#25199 was marked as a duplicate.

RDall10 months ago

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

comment:2 RDall10 months ago

  • Keywords has-patch added; needs-patch removed

obenland10 months ago

comment:3 lancewillett10 months ago

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

comment:4 lancewillett10 months 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.

comment:5 rdall10 months 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.