Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#13692 closed defect (bug) (fixed)

Twenty Ten: Enhancement to class name nopassword

Reported by: zeo Owned by:
Priority: normal Milestone: 3.0
Component: Themes Version:
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

Add the same padding value as id name comments-title so that we keep the gap consistency. see: class-nopassword-padding.diff and image attached.

Use p element instead of div for class name nopassword so that we keep the element consistency used by class name nocomments. Since both are wrapped by the same element - div#comments. see: class-nopassword-use-p.diff

Currently this is how it's structured:

<div id="comments">
   <div class="nopassword">...</div>
</div>
<div id="comments">
   <p class="nocomments">...</p>
</div>

Attachments (3)

nopassword.jpg​ (25.6 KB ) - added by zeo 16 years ago.
Comparison before and after clockwise.
class-nopassword-padding.diff​ (348 bytes ) - added by zeo 16 years ago.
Add the same padding value as id name comments-title.
class-nopassword-use-p.diff​ (696 bytes ) - added by zeo 16 years ago.
Use p element instead of div for class name nopassword.

Download all attachments as: .zip

Change History (6)

@zeo
16 years ago

Comparison before and after clockwise.

@zeo
16 years ago

Add the same padding value as id name comments-title.

@zeo
16 years ago

Use p element instead of div for class name nopassword.

#1 @nacin
16 years ago

  • Keywords commit added; 2nd-opinion removed

I see no problems with using <p>.

#2 @nacin
16 years ago

  • Resolution → fixed
  • Status new → closed

(In [15121]) Use p.nopassword instead of div.nopassword in Twenty Ten. props zeo, fixes #13692.

#3 @nacin
16 years ago

(In [15125]) Hide 'no password' text for comments. It's already being shown for the_content; aligns it with .nocomments; fixes #13692.

Note: See TracTickets for help on using tickets.