WordPress.org

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

#21242 closed defect (bug) (fixed)

Twenty Twelve: HTML elements outside of .entry-content need styling

Reported by: obenland Owned by: lancewillett
Milestone: 3.5 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords:
Focuses: Cc:

Description

Specifically the password form for password protected posts and HTML elements inside of comments.

Screenshots will be attached.

Attachments (3)

comment-text.png (149.7 KB) - added by obenland 9 years ago.
passwordform.png (23.7 KB) - added by obenland 9 years ago.
21242.diff (6.3 KB) - added by drewstrojny 9 years ago.

Download all attachments as: .zip

Change History (14)

@obenland
9 years ago

@obenland
9 years ago

#1 @SergeyBiryukov
9 years ago

  • Milestone changed from Awaiting Review to 3.5

@drewstrojny
9 years ago

#2 @drewstrojny
9 years ago

Attachment 21242.diff styles the necessary password protected post elements and HTML elements inside of comments. Thanks obenland!

#3 @drewstrojny
9 years ago

  • Cc dstrojny@… added

#4 @lancewillett
9 years ago

  • Cc drewstrojny added; dstrojny@… removed

#5 @lancewillett
9 years ago

  • Keywords needs-patch needs-ui removed

@drewstrojny Patch is looking good, will commit shortly. The only change is removing the "alignleft, alignright, aligncenter" rules, I don't think those apply to comment content.

#6 @lancewillett
9 years ago

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

In [21277]:

Twenty Twelve: add styles for common HTML elements in comment content, and also style up password protected form. Props drewstrojny, fixes #21242.

#7 @obenland
9 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Styles still don't get applied, since comment content is missing the 'comment-content' CSS class. Currently it has 'post-content'.

Should I submit a patch? :)

#8 follow-ups: @Jayjdk
9 years ago

What is the reason for using something like .entry-content pre, .comment-content pre instead of just pre ?

#9 in reply to: ↑ 8 @obenland
9 years ago

Replying to Jayjdk:

What is the reason for using something like .entry-content pre, .comment-content pre instead of just pre ?

Related: #21379 (Nr. 2)

#10 in reply to: ↑ 8 @lancewillett
9 years ago

Replying to Jayjdk:

What is the reason for using something like .entry-content pre, .comment-content pre instead of just pre ?

This is important so that plugin or widget output isn't styled, in the case of main content HTML elements we want to specifically focus on comment area and content area of posts and pages only.

#11 @lancewillett
9 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In [21371]:

Twenty Twelve: fix typo for comment-content container showing incorrect class value, props obenland. Closes #21242.

Note: See TracTickets for help on using tickets.