WordPress.org

Make WordPress Core

#23524 closed defect (bug) (fixed)

Twenty Thirteen: Comment reply form for replies via the 'Reply' link shrinks in wider viewports

Reported by: SriniG Owned by: lancewillett
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.6
Component: Bundled Theme Keywords: has-patch commit
Focuses: Cc:

Description

Comment reply form for replies to an existing comment via the 'Reply' link shrinks sideways in viewports wider than 1000px. And this happens only when there is an active sidebar. This condition does not occur in screen widths less than 1000px (where the sidebar falls down gracefully), or if there is no sidebar. Tested with Chromium Version 24.0.1312.56 and Firefox 18.0.2 in Ubuntu 12.04.

Attachments (3)

twentythirteen-page-with-comments-reply.png (44.8 KB) - added by SriniG 17 months ago.
comment-reply-form-fix.patch (483 bytes) - added by SriniG 17 months ago.
Patch that fixes the bug.
23524-fix.patch (415 bytes) - added by SriniG 17 months ago.
Per comment-3 by obenland.

Download all attachments as: .zip

Change History (9)

SriniG17 months ago

Patch that fixes the bug.

comment:1 SriniG17 months ago

  • Keywords has-patch needs-testing added

comment:2 lancewillett17 months ago

  • Milestone changed from Awaiting Review to 3.6

comment:3 follow-up: obenland17 months ago

  • Keywords needs-testing removed

We already have a fix for that in place (line 2309), the selector is just too weak. Adding #respond before #commentform does the trick.

Did I mention I wished there were classes in all of comment_form()?

comment:4 in reply to: ↑ 3 SriniG17 months ago

Yes, adding a #respond before #commentform in line 2309 solves it! Thanks.

SriniG17 months ago

Per comment-3 by obenland.

comment:5 lancewillett17 months ago

  • Keywords commit added

comment:6 lancewillett17 months ago

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

In 23462:

Twenty Thirteen: fix comment form layout for inline replies in wide viewports. Fixes #23524, props SriniG and obenland.

(Also fix an extra space that crept into r23461, see #23511.)

Note: See TracTickets for help on using tickets.