Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#21242 closed defect (bug) (fixed)

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

Reported by: obenland's profile obenland Owned by: lancewillett's profile 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 12 years ago.
passwordform.png (23.7 KB) - added by obenland 12 years ago.
21242.diff (6.3 KB) - added by drewstrojny 12 years ago.

Download all attachments as: .zip

Change History (14)

#1 @SergeyBiryukov
12 years ago

  • Milestone changed from Awaiting Review to 3.5

@drewstrojny
12 years ago

#2 @drewstrojny
12 years ago

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

#3 @drewstrojny
12 years ago

  • Cc dstrojny@… added

#4 @lancewillett
12 years ago

  • Cc drewstrojny added; dstrojny@… removed

#5 @lancewillett
12 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
12 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
12 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
12 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
12 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
12 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
12 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.