Opened 2 years ago

Closed 2 years ago

#17437 closed defect (bug) (fixed)

Twenty Eleven: use p.nopassword instead of div.nopassword

Reported by: zeo Owned by: iandstewart
Priority: normal Milestone: 3.2
Component: Themes Version: 3.2
Severity: normal Keywords: has-patch dev-feedback
Cc:

Description

In Twenty Eleven, use p.nopassword instead of div.nopassword to make it consistent with p.nocomments so that we can use its styling.

Also fixes some typo in Twenty Eleven comments.php

Attachments (1)

17437.diff​ (1.8 KB) - added by zeo 2 years ago.

Download all attachments as: .zip

Change History (6)

zeo — 2 years ago

comment:1   zeo — 2 years ago

  • Type changed from enhancement to defect (bug)

comment:2   zeo — 2 years ago

If we follow Twenty Ten, class .nopassword and .nocomments is display: none

comment:3   nacin — 2 years ago

  • Milestone changed from Awaiting Review to 3.2

comment:4   iandstewart — 2 years ago

This looks good to me. We shouldn't have to display:none .nocomments as it should only appear on posts and not pages.

comment:5   iandstewart — 2 years ago

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

In [18039]:

Twenty Eleven: consistent styles for .nopassword and .nocomments; Props zeo; Fixes #17437

Note: See TracTickets for help on using tickets.