Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#21242 closed defect (bug) (fixed)

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

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

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 14 years ago.
passwordform.png (23.7 KB ) - added by obenland 14 years ago.
21242.diff (6.3 KB ) - added by drewstrojny 14 years ago.

Download all attachments as: .zip

Change History (14)

#1 @SergeyBiryukov
14 years ago

  • Milestone Awaiting Review3.5

@drewstrojny
14 years ago

#2 @drewstrojny
14 years ago

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

#3 @drewstrojny
14 years ago

  • Cc dstrojny@… added

#4 @lancewillett
14 years ago

  • Cc drewstrojny added; dstrojny@… removed

#5 @lancewillett
14 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
14 years ago

  • Owner set to lancewillett
  • Resolutionfixed
  • Status newclosed

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
14 years ago

  • Resolution fixed
  • Status closedreopened

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
14 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
14 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
14 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
14 years ago

  • Resolutionfixed
  • Status reopenedclosed

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.